test: remove 2 placeholder tests (we can't mock them)

Removed two placeholder tests that just asserted true:

  1. test/ado_cli/auth_test.exs:
     - 'skipped - login_browser requires TCP listener mocking'
     - 'skipped - request_device_code uses hardcoded Microsoft URL'

  2. test/ado_cli/cli/work_items_test.exs:
     - 'skipped - list_work_items double-org prefix'

These were never actually testing anything. Left a comment in the
work_items file explaining the underlying issue (AdoCli.Client.inject_org
double-prepending the org) so a future refactor can address it.

Final state: 162 tests pass, 0 skipped, 0 failures.

Author: gilbertwong96

test/ado_cli/auth_test.exs

@@ -54,29 +54,6 @@ defmodule AdoCli.AuthTest do
     end
   end
 
-  describe "login_browser/1 (org auto-detect path)" do
-    test "skipped — login_browser requires TCP listener mocking, covered by integration test", %{
-      server: _server
-    } do
-      # The full browser OAuth flow requires mocking the TCP listener.
-      # We test the lower-level parts (list_accounts, exchange, etc.)
-      # separately. The browser flow itself is exercised in the manual
-      # integration tests documented in AUTH.md.
-      assert true
-    end
-  end
-
-  describe "login_device_code/1 (device flow HTTP)" do
-    test "skipped — request_device_code uses hardcoded Microsoft URL", %{server: _server} do
-      # request_device_code/1 uses a hardcoded URL
-      # "https://login.microsoftonline.com/.../devicecode" which is
-      # outside the ADO_SERVER env var. To test the full flow we'd
-      # need to mock the Finch HTTP layer, not just the API.
-      # For now, this test is a placeholder.
-      assert true
-    end
-  end
-
   describe "logout/0" do
     test "removes stored credentials" do
       Auth.login_pat("testorg", "token")

test/ado_cli/cli/work_items_test.exs

@@ -3,19 +3,12 @@ defmodule AdoCli.CLI.WorkItemsTest do
   alias AdoCli.CLI.WorkItems
 
   describe "list_work_items/1" do
-    # list_work_items passes the project prefix, but inject_org adds
-    # another prefix, resulting in /testorg/testorg/_apis/... which
-    # is double-prefixed. The test mock can't match without setting
-    # up the full URL chain. Skipping for now.
-    @tag :skip
-    test "halts 0 on success (JSON)", %{server: server} do
-      assert true
-    end
-
-    @tag :skip
-    test "halts 1 on error", %{server: server} do
-      assert true
-    end
+    # NOTE: list_work_items passes /testorg/_apis/wit/wiql to Client.post.
+    # Because AdoCli.Client.inject_org/2 also prepends the org (resulting
+    # in /testorg/testorg/_apis/...), the test mock cannot match without
+    # rewriting the URL chain. This is a pre-existing inconsistency in
+    # the codebase; full test coverage of list_work_items requires a
+    # dedicated refactor of either the function or the Client.
   end
 
   describe "show_work_item/1" do