Merge branch 'master' into fix-error-message

doc/tests.rst

@@ -127,7 +127,7 @@ following test elements are clearly identified:
 - the test expectation
 - the test outcome
 
-You can have a look at `this blob post
+You can have a look at `this blog post
 <https://blog.janestreet.com/testing-with-expectations/>`_ to find out
 more about expectation tests. But Back to Jbuilder, the workflow for
 expectation tests is always as follow:

test/unit-tests/sexp.mlt

@@ -107,3 +107,33 @@ parse {|"\%{x}"|}
 - : parse_result =
 Different {jbuild = Ok ["\\%{x}"]; dune = Error "unknown escape sequence"}
 |}]
+
+parse {|"$foo"|}
+[%%expect{|
+- : parse_result = Same (Ok ["$foo"])
+|}]
+
+parse {|"%foo"|}
+[%%expect{|
+- : parse_result = Same (Ok ["%foo"])
+|}]
+
+parse {|"bar%foo"|}
+[%%expect{|
+- : parse_result = Same (Ok ["bar%foo"])
+|}]
+
+parse {|"bar$foo"|}
+[%%expect{|
+- : parse_result = Same (Ok ["bar$foo"])
+|}]
+
+parse {|"%bar$foo%"|}
+[%%expect{|
+- : parse_result = Same (Ok ["%bar$foo%"])
+|}]
+
+parse {|"$bar%foo%"|}
+[%%expect{|
+- : parse_result = Same (Ok ["$bar%foo%"])
+|}]