Tambahkan opsi --lokasi-kuki untuk program CLI

Author: laymonage

README.md

@@ -274,6 +274,14 @@ Untuk menghapus kuki, gunakan opsi `--bersihkan` atau `-c`.
 $ kbbi-autentikasi --bersihkan
 ```
 
+Gunakan opsi `--lokasi-kuki` atau `-l` untuk menentukan lokasi kuki yang akan
+disimpan/dimuat.
+
+```
+$ kbbi-autentikasi $KBBI_POSEL $KBBI_SANDI --lokasi-kuki kukiku.json
+$ kbbi alam --lokasi-kuki kukiku.json
+```
+
 > **Catatan:**\
 > **`kbbi`** juga bisa dipanggil dengan **`python kbbi.py`**.\
 > **`kbbi-autentikasi`** juga bisa dipanggil dengan **`python -c "import kbbi; kbbi.autentikasi()"`**

src/kbbi/kbbi.py

@@ -500,7 +500,7 @@ def __init__(self, posel=None, sandi=None, lokasi_kuki=None):
 
         :param posel: Alamat posel yang terdaftar di KBBI Daring
         :type email: str
-        :param sandi: Kata sandi untuk akun dengan alamat surel yang diberikan
+        :param sandi: Kata sandi untuk akun dengan alamat posel yang diberikan
         :type sandi: str
         :param lokasi_kuki: Lokasi kuki yang akan dimuat/disimpan
         :type lokasi_kuki: str atau PathLike
@@ -511,10 +511,7 @@ def __init__(self, posel=None, sandi=None, lokasi_kuki=None):
             try:
                 self.ambil_kuki()
             except FileNotFoundError as e:
-                raise GagalAutentikasi(
-                    "Posel dan sandi tidak diberikan, "
-                    f"tetapi kuki tidak ditemukan di {self.lokasi_kuki}"
-                ) from e
+                raise KukiTidakDitemukan(self.lokasi_kuki) from e
         else:
             token = self._ambil_token()
             self._autentikasi(posel, sandi, token)
@@ -588,12 +585,24 @@ class GagalAutentikasi(Galat):
     """Galat ketika gagal melakukan autentikasi dengan KBBI."""
 
     def __init__(self, pesan=None):
-        if pesan is None:
-            pesan = (
-                "Gagal melakukan autentikasi dengan alamat posel dan sandi "
-                "yang diberikan."
+        super().__init__(
+            pesan
+            or "Gagal melakukan autentikasi dengan alamat posel dan sandi "
+            "yang diberikan."
+        )
+
+
+class KukiTidakDitemukan(GagalAutentikasi):
+    """Galat ketika lokasi kuki yang diberikan tidak ditemukan."""
+
+    def __init__(self, lokasi_kuki, posel_sandi=True):
+        if posel_sandi:
+            super().__init__(
+                f"Posel dan sandi tidak diberikan, "
+                f"tetapi kuki tidak ditemukan pada {lokasi_kuki}"
             )
-        super().__init__(pesan)
+        else:
+            super().__init__(f"Kuki tidak ditemukan pada {lokasi_kuki}!")
 
 
 def _parse_args_autentikasi(args):
@@ -618,13 +627,19 @@ def _parse_args_autentikasi(args):
         help="kata sandi akun KBBI Daring dengan posel yang diberikan",
         nargs="?",
     )
+    parser.add_argument(
+        "--lokasi-kuki",
+        "-l",
+        help="lokasi menuju berkas kuki yang akan disimpan",
+        metavar="LOKASI",
+    )
     parser.add_argument(
         "-h",
         "-b",
         "--help",
         "--bantuan",
-        action="help",
-        default=argparse.SUPPRESS,
+        action="store_true",
+        dest="bantuan",
         help="tampilkan pesan bantuan ini dan keluar",
     )
     parser.add_argument(
@@ -633,15 +648,14 @@ def _parse_args_autentikasi(args):
         help="bersihkan kuki yang tersimpan",
         action="store_true",
     )
-    return parser.parse_args(args)
+    return parser.parse_args(args), parser
 
 
-def _bersihkan_kuki():
-    lokasi_kuki = AutentikasiKBBI.lokasi_kuki
+def _bersihkan_kuki(lokasi_kuki):
     try:
         lokasi_kuki.unlink()
     except FileNotFoundError:
-        print(f"Kuki tidak ditemukan pada {lokasi_kuki}!")
+        print(KukiTidakDitemukan(lokasi_kuki, posel_sandi=False))
         return 1
     else:
         print(f"Kuki {lokasi_kuki} berhasil dihapus.")
@@ -652,15 +666,21 @@ def autentikasi(argv=None):
     """Program CLI untuk autentikasi."""
     if argv is None:
         argv = sys.argv[1:]
-    if argv == []:
-        argv = ["-h"]
-    args = _parse_args_autentikasi(argv)
+    args, parser = _parse_args_autentikasi(argv)
+    lokasi_kuki = AutentikasiKBBI.lokasi_kuki
+    if args.lokasi_kuki:
+        lokasi_kuki = Path(args.lokasi_kuki)
     if args.posel is None and args.sandi is None:
         if args.bersihkan:
-            return _bersihkan_kuki()
+            return _bersihkan_kuki(lokasi_kuki)
+        args.bantuan = True
+    if args.bantuan:
+        parser.print_help()
         return 0
     try:
-        auth = AutentikasiKBBI(args.posel, args.sandi)
+        auth = AutentikasiKBBI(
+            args.posel, args.sandi, lokasi_kuki=args.lokasi_kuki
+        )
     except Galat as e:
         print(e)
         return 1
@@ -671,6 +691,10 @@ def autentikasi(argv=None):
             f"{auth.lokasi_kuki}.\n"
             "Kuki akan otomatis digunakan pada penggunaan KBBI berikutnya."
         )
+        if args.lokasi_kuki:
+            print(
+                "Gunakan opsi --lokasi-kuki yang sama ketika menggunakan KBBI."
+            )
     return 0
 
 
@@ -728,6 +752,12 @@ def _parse_args_utama(args):
         type=int,
         metavar="N",
     )
+    parser.add_argument(
+        "--lokasi-kuki",
+        "-l",
+        help="lokasi menuju berkas kuki yang akan digunakan untuk autentikasi",
+        metavar="L",
+    )
     return parser.parse_args(args)
 
 
@@ -746,8 +776,14 @@ def main(argv=None):
         argv = sys.argv[1:]
     args = _parse_args_utama(argv)
     auth = None
-    if AutentikasiKBBI.lokasi_kuki.exists():
-        auth = AutentikasiKBBI()
+    lokasi_kuki = AutentikasiKBBI.lokasi_kuki
+    if args.lokasi_kuki:
+        lokasi_kuki = Path(args.lokasi_kuki)
+    if lokasi_kuki.exists():
+        auth = AutentikasiKBBI(lokasi_kuki=lokasi_kuki)
+    elif args.lokasi_kuki:
+        print(KukiTidakDitemukan(lokasi_kuki, posel_sandi=False))
+        return 1
     try:
         laman = KBBI(args.laman, auth)
     except Galat as e:

tests/conftest.py

@@ -99,18 +99,22 @@ def autentikasi_sukses(request, monkeypatch):
 
 
 @pytest.fixture
-def lokasi_kuki(monkeypatch):
-    lokasi = pathlib.Path("kukifix.json")
-    monkeypatch.setattr(kbbi.AutentikasiKBBI, "lokasi_kuki", lokasi)
-    return lokasi
+def lokasi_kuki():
+    return pathlib.Path("kukifix.json")
 
 
 @pytest.fixture
-def kuki(lokasi_kuki):
-    lokasi_kuki.write_text('{".AspNet.ApplicationCookie": "kuki enak"}')
+def mock_lokasi_kuki(lokasi_kuki, monkeypatch):
+    monkeypatch.setattr(kbbi.AutentikasiKBBI, "lokasi_kuki", lokasi_kuki)
     return lokasi_kuki
 
 
+@pytest.fixture
+def kuki(mock_lokasi_kuki):
+    mock_lokasi_kuki.write_text('{".AspNet.ApplicationCookie": "kuki enak"}')
+    return mock_lokasi_kuki
+
+
 @pytest.fixture
 def tanpa_kuki(lokasi_kuki):
     if lokasi_kuki.exists():

tests/test_cli.py

@@ -5,7 +5,7 @@
 import kbbi
 
 
-def test_bersihkan_kuki_tidak_ada(tanpa_kuki, capsys):
+def test_bersihkan_kuki_tidak_ada(mock_lokasi_kuki, tanpa_kuki, capsys):
     hasil = kbbi.autentikasi(["--bersihkan"])
     tangkap = capsys.readouterr()
     assert tangkap.out == "Kuki tidak ditemukan pada kukifix.json!\n"
@@ -41,14 +41,34 @@ def test_autentikasi_gagal(autentikasi_gagal, capsys):
     assert hasil == 1
 
 
-def test_autentikasi_sukses(autentikasi_sukses, lokasi_kuki, capsys):
+def test_autentikasi_sukses(autentikasi_sukses, mock_lokasi_kuki, capsys):
     hasil = kbbi.autentikasi(["[email protected]", "p4sti_sukses"])
     tangkap = capsys.readouterr()
     assert tangkap.out == (
         "Autentikasi berhasil dan kuki telah disimpan di kukifix.json.\n"
         "Kuki akan otomatis digunakan pada penggunaan KBBI berikutnya.\n"
     )
     assert hasil == 0
+    assert (
+        mock_lokasi_kuki.read_text()
+        == '{".AspNet.ApplicationCookie": "mockcookie"}'
+    )
+    mock_lokasi_kuki.unlink()
+
+
+def test_autentikasi_sukses_dengan_lokasi_kuki(
+    autentikasi_sukses, lokasi_kuki, capsys
+):
+    hasil = kbbi.autentikasi(
+        ["[email protected]", "p4sti_sukses", "--lokasi-kuki", "kukifix.json"]
+    )
+    tangkap = capsys.readouterr()
+    assert tangkap.out == (
+        "Autentikasi berhasil dan kuki telah disimpan di kukifix.json.\n"
+        "Kuki akan otomatis digunakan pada penggunaan KBBI berikutnya.\n"
+        "Gunakan opsi --lokasi-kuki yang sama ketika menggunakan KBBI.\n"
+    )
+    assert hasil == 0
     assert (
         lokasi_kuki.read_text()
         == '{".AspNet.ApplicationCookie": "mockcookie"}'
@@ -171,3 +191,11 @@ def test_program_utama_main(
     tangkap = capsys.readouterr()
     assert tangkap.out == lokasi.read_text()
     assert hasil == 0
+
+
+@pytest.mark.parametrize("kbbi_mock", [None], indirect=True)
+def test_program_utama_lokasi_kuki_tidak_ada(capsys, kbbi_mock, tanpa_kuki):
+    hasil = kbbi.main(["alam", "--lokasi-kuki", "kukiku.json"])
+    tangkap = capsys.readouterr()
+    assert tangkap.out == "Kuki tidak ditemukan pada kukiku.json!\n"
+    assert hasil == 1

tests/test_kbbi.py

@@ -41,7 +41,7 @@ def test_autentikasi_tanpa_posel_sandi_kuki_tidak_ada():
         kbbi.AutentikasiKBBI(lokasi_kuki="/lokasi/tidak/ada")
     assert str(e.value) == (
         "Posel dan sandi tidak diberikan, "
-        "tetapi kuki tidak ditemukan di /lokasi/tidak/ada"
+        "tetapi kuki tidak ditemukan pada /lokasi/tidak/ada"
     )