Skip to content

Commit b318d1f

Browse files
committed
test: remove pedantic tests
1 parent 2fde3ff commit b318d1f

File tree

2 files changed

+1
-63
lines changed

2 files changed

+1
-63
lines changed

ibind/oauth/__init__.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ def version(self):
2828
"""
2929
raise NotImplementedError()
3030

31-
def verify_config(self):
31+
def verify_config(self): # pragma: no cover
3232
return
3333

3434
init_oauth: bool = var.IBIND_INIT_OAUTH

test/unit/oauth/test_oauth_base_config_u.py

Lines changed: 0 additions & 62 deletions
Original file line numberDiff line numberDiff line change
@@ -50,18 +50,6 @@ def test_verify_config_base_implementation(concrete_config):
5050
assert result is None
5151

5252

53-
def test_oauth_config_default_attributes():
54-
# Arrange & Act
55-
config = ConcreteOAuthConfig()
56-
57-
# Assert
58-
# Test that default values are set (these come from var module)
59-
assert hasattr(config, 'init_oauth')
60-
assert hasattr(config, 'init_brokerage_session')
61-
assert hasattr(config, 'maintain_oauth')
62-
assert hasattr(config, 'shutdown_oauth')
63-
64-
6553
def test_copy_method_creates_shallow_copy(concrete_config):
6654
# Arrange
6755
original_id = id(concrete_config)
@@ -122,53 +110,3 @@ def test_copy_method_with_multiple_modifications(concrete_config):
122110
assert getattr(copied_config, attr) == expected_value
123111

124112

125-
def test_copy_preserves_type(concrete_config):
126-
# Arrange
127-
128-
# Act
129-
copied_config = concrete_config.copy()
130-
131-
# Assert
132-
assert type(copied_config) is type(concrete_config)
133-
assert isinstance(copied_config, ConcreteOAuthConfig)
134-
assert isinstance(copied_config, OAuthConfig)
135-
136-
137-
def test_copy_method_with_no_modifications(concrete_config):
138-
# Arrange
139-
140-
# Act
141-
copied_config = concrete_config.copy()
142-
143-
# Assert
144-
# All attributes should be identical
145-
assert copied_config.init_oauth == concrete_config.init_oauth
146-
assert copied_config.init_brokerage_session == concrete_config.init_brokerage_session
147-
assert copied_config.maintain_oauth == concrete_config.maintain_oauth
148-
assert copied_config.shutdown_oauth == concrete_config.shutdown_oauth
149-
# But should be a different object
150-
assert copied_config is not concrete_config
151-
152-
153-
def test_default_values_are_set():
154-
# Arrange & Act
155-
config = ConcreteOAuthConfig()
156-
157-
# Assert
158-
# Test that all required attributes exist with boolean values
159-
assert isinstance(config.init_oauth, bool)
160-
assert isinstance(config.init_brokerage_session, bool)
161-
assert isinstance(config.maintain_oauth, bool)
162-
assert isinstance(config.shutdown_oauth, bool)
163-
164-
165-
def test_copy_method_edge_case_empty_kwargs(concrete_config):
166-
# Arrange
167-
empty_kwargs = {}
168-
169-
# Act
170-
copied_config = concrete_config.copy(**empty_kwargs)
171-
172-
# Assert
173-
assert copied_config is not concrete_config
174-
assert copied_config.init_oauth == concrete_config.init_oauth

0 commit comments

Comments
 (0)