session/test_env_select: Reverse-engineer CliEnv and add tests for it

Some of the behaviour here, such as treatment of zero-length environment
names specified by the user, is not really intentional, but apparently an
artifact of the method used to parse the environment name list supplied by
the user.

Author: 0cjs

tests/session/test_env_select.py

@@ -19,6 +19,41 @@
 #   Tests directly on env_select functionality.
 
 
+@pytest.mark.parametrize(
+    ("userinput",       "envnames",             "isall",    "isdefault" ), [
+    (None,              (),                     False,      True        ),
+    ("",                (),                     False,      True        ),
+    ("a1",              ("a1",),                False,      False       ),
+    ("a1,b2,c3",        ("a1", "b2", "c3"),     False,      False       ),
+    #   If the user gives "ALL" as any envname, this becomes an "is_all" and other envnames are ignored.
+    ("ALL",             (),                     True,       False       ),
+    ("a1,ALL,b2",       (),                     True,       False       ),
+    #   Zero-length envnames are ignored as being not present. This is not intentional.
+    (",,a1,,,b2,,",     ("a1", "b2"),           False,      False       ),
+    (",,",              (),                     False,      True        ),
+    #   Environment names with "invalid" characters are accepted here; the client is expected to deal with this.
+    ("\x01.-@\x02,xxx", ("\x01.-@\x02", "xxx"), False,      False       ),
+    ],
+)
+def test_clienv(userinput: str, envnames: tuple[str], isall: bool, isdefault: bool) -> None:
+    c = CliEnv(userinput)
+    assert (c.is_all, c.is_default_list, tuple(c)) \
+        == (isall,    isdefault,         tuple(envnames))
+
+@pytest.mark.parametrize(
+    ("userinput", "expected"),
+    [
+        ("", False),
+        ("all", False),
+        ("All", False),
+        ("ALL", True),
+        ("a,ALL,b", True),
+    ],
+)
+def test_clienv_is_all(userinput: str, expected: bool) -> None:
+    assert CliEnv(userinput).is_all is expected
+
+
 def test_env_select_lazily_looks_at_envs() -> None:
     state = State(get_options(), [])
     env_selector = EnvSelector(state)