fix(context): Reduce scope of public API

BREAKING CHANGE: `Context::get_val` is no longer public.  Use
`Context::get_val_by_index` instead.

Author: epage

liquid-interpreter/src/context.rs

@@ -140,16 +140,6 @@ impl<'g> Stack<'g> {
         };
     }
 
-    /// Gets a value from the rendering context.
-    pub fn get_val<'a>(&'a self, name: &str) -> Option<&'a Value> {
-        for frame in self.stack.iter().rev() {
-            if let rval @ Some(_) = frame.get(name) {
-                return rval;
-            }
-        }
-        self.globals.and_then(|g| g.get(name))
-    }
-
     /// Recursively index into the stack.
     pub fn get_val_by_index<'i, I: Iterator<Item = &'i Index>>(
         &self,
@@ -174,6 +164,15 @@ impl<'g> Stack<'g> {
         })
     }
 
+    fn get_val<'a>(&'a self, name: &str) -> Option<&'a Value> {
+        for frame in self.stack.iter().rev() {
+            if let rval @ Some(_) = frame.get(name) {
+                return rval;
+            }
+        }
+        self.globals.and_then(|g| g.get(name))
+    }
+
     /// Sets a value in the global context.
     pub fn set_global_val<S>(&mut self, name: S, val: Value) -> Option<Value>
     where

src/tags/assign_tag.rs

@@ -57,6 +57,7 @@ mod test {
     use interpreter;
     use tags;
     use value::Value;
+    use value::Index;
 
     fn options() -> LiquidOptions {
         let mut options = LiquidOptions::default();
@@ -103,8 +104,8 @@ mod test {
 
             let output = template.render(&mut context).unwrap();
             assert_eq!(
-                context.stack().get_val("freestyle"),
-                Some(&Value::scalar(false))
+                context.stack().get_val_by_index([Index::with_key("freestyle")].iter()).unwrap(),
+                &Value::scalar(false)
             );
             assert_eq!(output, "");
         }
@@ -124,8 +125,8 @@ mod test {
 
             let output = template.render(&mut context).unwrap();
             assert_eq!(
-                context.stack().get_val("freestyle"),
-                Some(&Value::scalar(true))
+                context.stack().get_val_by_index([Index::with_key("freestyle")].iter()).unwrap(),
+                &Value::scalar(true)
             );
             assert_eq!(output, "<p>Freestyle!</p>");
         }

src/tags/capture_block.rs

@@ -66,6 +66,7 @@ mod test {
     use super::*;
     use compiler;
     use interpreter;
+    use value::Index;
 
     fn options() -> LiquidOptions {
         let mut options = LiquidOptions::default();
@@ -95,8 +96,8 @@ mod test {
 
         let output = template.render(&mut ctx).unwrap();
         assert_eq!(
-            ctx.stack().get_val("attribute_name"),
-            Some(&Value::scalar("potato-42-color"))
+            ctx.stack().get_val_by_index([Index::with_key("attribute_name")].iter()).unwrap(),
+            &Value::scalar("potato-42-color")
         );
         assert_eq!(output, "");
     }