Skip to content

Commit 1ac7754

Browse files
committed
Refactor integration tests for improved readability and maintainability
- Reorganized import statements for clarity. - Enhanced formatting of pytest.skipif decorator for better readability. - Streamlined test methods by removing unnecessary comments and whitespace. - Improved error handling in tests to ensure proper cleanup. - Updated test assertions for consistency and clarity. - Removed deprecated performance tests to streamline test suite. - Enhanced security tests to cover additional edge cases and improve validation. - Improved test coverage for NocoDBTable methods, ensuring proper mocking and assertions.
1 parent ee8379a commit 1ac7754

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

59 files changed

+1853
-1952
lines changed

.github/CODEOWNERS

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,4 +13,4 @@
1313
# Default rule: assign all files (*) in the repository to @karlspace.
1414
# This means every pull request will automatically request a review from you,
1515
# unless a more specific rule matches first.
16-
* @karlspace
16+
* @karlspace

.github/config/pr-labeler/path-labels.yml

Lines changed: 129 additions & 129 deletions
Original file line numberDiff line numberDiff line change
@@ -13,176 +13,176 @@ size-labels:
1313
# Path-based labeling rules
1414
path-labels:
1515
# Source code changes
16-
'src/nocodb_simple_client/':
17-
- 'area: core'
18-
- 'type: enhancement'
19-
20-
'src/nocodb_simple_client/client.py':
21-
- 'area: client'
22-
- 'component: api'
23-
24-
'src/nocodb_simple_client/table.py':
25-
- 'area: table'
26-
- 'component: table-operations'
27-
28-
'src/nocodb_simple_client/exceptions.py':
29-
- 'area: exceptions'
30-
- 'component: error-handling'
31-
16+
"src/nocodb_simple_client/":
17+
- "area: core"
18+
- "type: enhancement"
19+
20+
"src/nocodb_simple_client/client.py":
21+
- "area: client"
22+
- "component: api"
23+
24+
"src/nocodb_simple_client/table.py":
25+
- "area: table"
26+
- "component: table-operations"
27+
28+
"src/nocodb_simple_client/exceptions.py":
29+
- "area: exceptions"
30+
- "component: error-handling"
31+
3232
# Test changes
33-
'tests/':
34-
- 'area: tests'
35-
- 'type: testing'
36-
37-
'tests/test_client.py':
38-
- 'area: tests'
39-
- 'component: client-tests'
40-
41-
'tests/test_table.py':
42-
- 'area: tests'
43-
- 'component: table-tests'
44-
45-
'tests/test_integration.py':
46-
- 'area: tests'
47-
- 'type: integration'
48-
- 'priority: high'
49-
33+
"tests/":
34+
- "area: tests"
35+
- "type: testing"
36+
37+
"tests/test_client.py":
38+
- "area: tests"
39+
- "component: client-tests"
40+
41+
"tests/test_table.py":
42+
- "area: tests"
43+
- "component: table-tests"
44+
45+
"tests/test_integration.py":
46+
- "area: tests"
47+
- "type: integration"
48+
- "priority: high"
49+
5050
# Documentation changes
51-
'README.MD':
52-
- 'area: documentation'
53-
- 'type: documentation'
54-
55-
'docs/':
56-
- 'area: documentation'
57-
- 'type: documentation'
58-
59-
'CHANGELOG.MD':
60-
- 'area: documentation'
61-
- 'type: changelog'
62-
51+
"README.MD":
52+
- "area: documentation"
53+
- "type: documentation"
54+
55+
"docs/":
56+
- "area: documentation"
57+
- "type: documentation"
58+
59+
"CHANGELOG.MD":
60+
- "area: documentation"
61+
- "type: changelog"
62+
6363
# Examples
64-
'examples/':
65-
- 'area: examples'
66-
- 'type: documentation'
67-
64+
"examples/":
65+
- "area: examples"
66+
- "type: documentation"
67+
6868
# Configuration changes
69-
'pyproject.toml':
70-
- 'area: build'
71-
- 'type: build'
72-
- 'priority: medium'
73-
74-
'.github/workflows/':
75-
- 'area: ci/cd'
76-
- 'type: ci'
77-
- 'priority: medium'
78-
79-
'.github/workflows/ci-cd.yml':
80-
- 'area: ci/cd'
81-
- 'type: ci'
82-
- 'priority: high'
83-
84-
'.github/workflows/release.yml':
85-
- 'area: ci/cd'
86-
- 'type: release'
87-
- 'priority: high'
88-
69+
"pyproject.toml":
70+
- "area: build"
71+
- "type: build"
72+
- "priority: medium"
73+
74+
".github/workflows/":
75+
- "area: ci/cd"
76+
- "type: ci"
77+
- "priority: medium"
78+
79+
".github/workflows/ci-cd.yml":
80+
- "area: ci/cd"
81+
- "type: ci"
82+
- "priority: high"
83+
84+
".github/workflows/release.yml":
85+
- "area: ci/cd"
86+
- "type: release"
87+
- "priority: high"
88+
8989
# Security files
90-
'SECURITY.MD':
91-
- 'area: security'
92-
- 'type: security'
93-
- 'priority: high'
94-
95-
'.github/workflows/security.yml':
96-
- 'area: security'
97-
- 'type: security'
98-
- 'priority: high'
90+
"SECURITY.MD":
91+
- "area: security"
92+
- "type: security"
93+
- "priority: high"
94+
95+
".github/workflows/security.yml":
96+
- "area: security"
97+
- "type: security"
98+
- "priority: high"
9999

100100
# Branch-based labeling
101101
branch-labels:
102102
feature/*:
103-
- 'type: feature'
103+
- "type: feature"
104104
fix/*:
105-
- 'type: bugfix'
105+
- "type: bugfix"
106106
hotfix/*:
107-
- 'type: hotfix'
108-
- 'priority: critical'
107+
- "type: hotfix"
108+
- "priority: critical"
109109
chore/*:
110-
- 'type: chore'
110+
- "type: chore"
111111
docs/*:
112-
- 'type: documentation'
112+
- "type: documentation"
113113
test/*:
114-
- 'type: testing'
114+
- "type: testing"
115115
ci/*:
116-
- 'type: ci'
116+
- "type: ci"
117117
security/*:
118-
- 'type: security'
119-
- 'priority: high'
118+
- "type: security"
119+
- "priority: high"
120120
refactor/*:
121-
- 'type: refactor'
121+
- "type: refactor"
122122

123123
# Title-based labeling (regex patterns)
124124
title-labels:
125-
- pattern: '(?i)(breaking|major)'
125+
- pattern: "(?i)(breaking|major)"
126126
labels:
127-
- 'type: breaking-change'
128-
- 'priority: critical'
129-
130-
- pattern: '(?i)(fix|bug)'
127+
- "type: breaking-change"
128+
- "priority: critical"
129+
130+
- pattern: "(?i)(fix|bug)"
131131
labels:
132-
- 'type: bugfix'
133-
134-
- pattern: '(?i)(feat|feature)'
132+
- "type: bugfix"
133+
134+
- pattern: "(?i)(feat|feature)"
135135
labels:
136-
- 'type: feature'
137-
138-
- pattern: '(?i)(doc|documentation)'
136+
- "type: feature"
137+
138+
- pattern: "(?i)(doc|documentation)"
139139
labels:
140-
- 'type: documentation'
141-
142-
- pattern: '(?i)(test|testing)'
140+
- "type: documentation"
141+
142+
- pattern: "(?i)(test|testing)"
143143
labels:
144-
- 'type: testing'
145-
146-
- pattern: '(?i)(security|vulnerability|cve)'
144+
- "type: testing"
145+
146+
- pattern: "(?i)(security|vulnerability|cve)"
147147
labels:
148-
- 'type: security'
149-
- 'priority: high'
150-
151-
- pattern: '(?i)(performance|perf|optimization)'
148+
- "type: security"
149+
- "priority: high"
150+
151+
- pattern: "(?i)(performance|perf|optimization)"
152152
labels:
153-
- 'type: performance'
154-
155-
- pattern: '(?i)(refactor|cleanup)'
153+
- "type: performance"
154+
155+
- pattern: "(?i)(refactor|cleanup)"
156156
labels:
157-
- 'type: refactor'
158-
159-
- pattern: '(?i)(ci|workflow|github actions)'
157+
- "type: refactor"
158+
159+
- pattern: "(?i)(ci|workflow|github actions)"
160160
labels:
161-
- 'type: ci'
161+
- "type: ci"
162162

163163
# Auto-reviewers based on areas
164164
auto-reviewers:
165-
'area: core':
165+
"area: core":
166166
reviewers:
167-
- 'karl.bauer@bauer-group.com'
168-
169-
'type: security':
167+
- "karl.bauer@bauer-group.com"
168+
169+
"type: security":
170170
reviewers:
171-
- 'karl.bauer@bauer-group.com'
171+
- "karl.bauer@bauer-group.com"
172172
review-required: true
173-
174-
'type: breaking-change':
173+
174+
"type: breaking-change":
175175
reviewers:
176-
- 'karl.bauer@bauer-group.com'
176+
- "karl.bauer@bauer-group.com"
177177
review-required: true
178178

179179
# Special handling
180180
special-labels:
181181
draft:
182-
- 'status: draft'
183-
182+
- "status: draft"
183+
184184
ready-for-review:
185185
remove:
186-
- 'status: draft'
186+
- "status: draft"
187187
add:
188-
- 'status: ready-for-review'
188+
- "status: ready-for-review"

0 commit comments

Comments
 (0)