← Back to issue list

tests: Fix name registration window limit test to latest changes

View original Github issue

Metadata

Project
snapcraft
Number
#1166
Type
pull request
State
closed
Author
fgallina
Labels
Created
Updated
Closed

Current evaluation

Closed and superseded by #1224. Contributors discussed improving the name registration limit test to prevent blocking, suggesting retry logic and counters. The author deferred to priority work, and the branch was closed in favor of the alternative PR.

Suggested action:

No scores available.

Issue body

Now it's possible to register N names (currently 10) during the course of a time window (currently 10 minutes). The integration test checking for the limit now reflects that.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Closed and superseded by #1224. Contributors discussed improving the name registration limit test to prevent blocking, suggesting retry logic and counters. The author deferred to priority work, and the branch was closed in favor of the alternative PR.
qwen3.6-35b-a3b-mtp-q6 Closed in favor of PR #1224. The changes updated an integration test for new name registration window limits, but discussions highlighted test blocking issues. The work was ultimately superseded by another branch.
qwen3.6-35b-a3b-mtp-q6 Closed without merging. Discussion highlighted test wait time performance concerns. The changes were ultimately superseded and closed in favor of PR #1224.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1224 tests: update name registration window limit test. snapcraft merged Merged after updating the name registration window limit test. The store test suite was split to resolve Travis timeouts, and the branch was updated per reviewer feedback. Addresses LP #1676725 and depends on PR #1225.
78%