Fix compiling of strict lookup in compat mode

Fixes #2149

Author: theodorejb

lib/handlebars/compiler/javascript-compiler.js

@@ -1175,7 +1175,9 @@ function strictLookup(requireTerminal, compiler, parts, startPartIndex, type) {
     stack = compiler.nameLookup(stack, parts[i], type);
   }
 
-  if (requireTerminal) {
+  // If compat mode already consumed all parts via depthedLookup, skip the strict terminal check
+  // since the stack already holds the resolved value, not an object to look the property up on.
+  if (requireTerminal && startPartIndex <= len) {
     return [
       compiler.aliasable('container.strict'),
       '(',

spec/strict.js

@@ -121,6 +121,22 @@ describe('strict', function () {
     });
   });
 
+  describe('strict and compat mode', function () {
+    it('GH-2149: should render correctly when block context is a boolean', function () {
+      expectTemplate('{{#foo}}Hello {{bar}}{{/foo}}')
+        .withCompileOptions({ strict: true, compat: true })
+        .withInput({ foo: true, bar: 'World' })
+        .toCompileTo('Hello World');
+    });
+
+    it('GH-2149: should render correctly when looking up a property on each item', function () {
+      expectTemplate('{{#each items}}{{name}}{{/each}}')
+        .withCompileOptions({ strict: true, compat: true })
+        .withInput({ items: [{ name: 'Hello' }] })
+        .toCompileTo('Hello');
+    });
+  });
+
   describe('assume objects', function () {
     it('should ignore missing property', function () {
       expectTemplate('{{hello}}')