Merge pull request #49 from 1Password/andi/prevent-ide-errors

Fix IDE errors around exposing items and secrets APIs

Author: AndyTitu

example/example.py

@@ -1,7 +1,6 @@
 import asyncio
 import os
-from onepassword.client import Client
-from onepassword.types import Item, ItemField, ItemSection
+from onepassword import *
 
 
 async def main():

src/onepassword/__init__.py

@@ -1,5 +1,7 @@
+# AUTO-GENERATED
 from .client import Client
 from .defaults import DEFAULT_INTEGRATION_NAME, DEFAULT_INTEGRATION_VERSION
+from .types import * # noqa F403
 from .secrets import Secrets
 from .items import Items
 
@@ -11,3 +13,6 @@
     "DEFAULT_INTEGRATION_NAME",
     "DEFAULT_INTEGRATION_VERSION",
 ]
+
+if hasattr(types, "__all__"):
+    __all__ += types.__all__

src/onepassword/client.py

@@ -1,3 +1,4 @@
+# AUTO-GENERATED
 import weakref
 from .core import _init_client, _release_client
 from .defaults import new_default_config
@@ -6,19 +7,23 @@
 
 
 class Client:
-    @classmethod
-    async def authenticate(cls, auth, integration_name, integration_version):
-        self = cls()
+	secrets: Secrets
+	items: Items
+	
 
-        self.config = new_default_config(
-            auth=auth,
-            integration_name=integration_name,
-            integration_version=integration_version,
-        )
-        client_id = int(await _init_client(self.config))
+	@classmethod
+	async def authenticate(cls, auth, integration_name, integration_version):
+		config = new_default_config(
+			auth=auth,
+			integration_name=integration_name,
+			integration_version=integration_version,
+		)
+		client_id = int(await _init_client(config))
 
-        self.secrets = Secrets(client_id)
-        self.items = Items(client_id)
-        self._finalizer = weakref.finalize(self, _release_client, client_id)
+		authenticated_client = cls()
 
-        return self
+		authenticated_client.secrets = Secrets(client_id)
+		authenticated_client.items = Items(client_id)
+		authenticated_client._finalizer = weakref.finalize(cls, _release_client, client_id)
+
+		return authenticated_client

src/onepassword/test_client.py

@@ -40,26 +40,6 @@ async def test_invalid_resolve():
 
 ## test client constructor
 
-
-# valid
-@pytest.mark.asyncio
-async def test_good_client_construction():
-    client = await onepassword.Client.authenticate(
-        auth=TOKEN,
-        integration_name=onepassword_defaults.DEFAULT_INTEGRATION_NAME,
-        integration_version=onepassword_defaults.DEFAULT_INTEGRATION_VERSION,
-    )
-    assert client.config["serviceAccountToken"] == TOKEN
-    assert (
-        client.config["integrationName"]
-        == onepassword_defaults.DEFAULT_INTEGRATION_NAME
-    )
-    assert (
-        client.config["integrationVersion"]
-        == onepassword_defaults.DEFAULT_INTEGRATION_VERSION
-    )
-
-
 # invalid
 @pytest.mark.asyncio
 async def test_client_construction_no_auth():