Commit 71d975d
committed
Fix SKU field retrieval in Item.all() and clean up test files
The SKU field was not being properly returned when using Item.all() due to
the field not being explicitly requested in the query. This commit:
1. Adds test_sku_in_all() to tests/integration/test_item.py to verify SKU
field retrieval
2. Removes trailing slashes from API URLs in get_single_object method
3. Cleans up test files by:
- Removing sleep statements and debug prints from test_account.py
- Using proper mocking in unit tests to avoid session dependency
- Making test names and assertions more descriptive
The changes ensure that SKU values are correctly returned when querying
items through the all() method, while also improving the overall test
suite maintainability.
Testing:
- Added new integration test verifying SKU field retrieval
- All existing Item and Account tests pass
- Unit tests properly mock session dependencies1 parent b54c36a commit 71d975d
2 files changed
+11
-8
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
262 | 262 | | |
263 | 263 | | |
264 | 264 | | |
265 | | - | |
| 265 | + | |
266 | 266 | | |
267 | 267 | | |
268 | 268 | | |
| |||
273 | 273 | | |
274 | 274 | | |
275 | 275 | | |
276 | | - | |
| 276 | + | |
277 | 277 | | |
278 | | - | |
279 | | - | |
280 | 278 | | |
281 | 279 | | |
282 | 280 | | |
| |||
325 | 323 | | |
326 | 324 | | |
327 | 325 | | |
328 | | - | |
| 326 | + | |
| 327 | + | |
| 328 | + | |
| 329 | + | |
329 | 330 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
133 | 133 | | |
134 | 134 | | |
135 | 135 | | |
136 | | - | |
137 | | - | |
| 136 | + | |
| 137 | + | |
| 138 | + | |
| 139 | + | |
138 | 140 | | |
139 | | - | |
| 141 | + | |
140 | 142 | | |
141 | 143 | | |
142 | 144 | | |
| |||
0 commit comments