Repository navigation
test(integration): allow an optional B2_REALM for the live tests - #367
sophiecarreras wants to merge 2 commits into
Conversation
Quality Keeper — Advisory ReviewSource:
Quality Keeper assessed the repository, executed checks, and collected results, covering 19 quality expectations. Five repository-quality areasHard expectations keep their exact Evidence State. Advisory applicability questions are separate. Implementation or suite presence is not proof of a trusted run.
Observability — 0 hard, 0 advisoryCan operators understand and diagnose repository-owned runtime behavior? Hard expectations None selected. Applicability Review Gaps (advisory) None. Product Analytics — 0 hard, 0 advisoryDoes repository-owned product analytics instrumentation behave safely and as declared? Hard expectations None selected. Applicability Review Gaps (advisory) None. User Feedback — 0 hard, 0 advisoryDo repository-owned feedback and support-intake flows behave safely and accessibly? Hard expectations None selected. Applicability Review Gaps (advisory) None. Reliability — 3 hard, 0 advisoryDoes repository-owned software fail, recover, time out, and report health predictably? Hard expectations
Applicability Review Gaps (advisory) None. Security — 4 hard, 0 advisoryDo static controls and repository-owned security behaviors protect the intended boundaries? Hard expectations
Applicability Review Gaps (advisory) None. Other registered quality — 12 hard, 0 advisoryRegistered Testing Types not assigned to one of the five focused areas. Hard expectations
Applicability Review Gaps (advisory) None. Result
CoveredQuality Keeper found acceptable evidence for one or more expectations in these Testing Types.
Needs attentionQuality Keeper does not yet have acceptable evidence for these Testing Types. Some have partial checks that need to run, be fixed, or be connected. Others need new proof or seeded tests. In both cases, Quality Keeper cannot count them as covered yet.
Quality Keeper changes no Evidence State. Pending, missing, executed-but-not-exercised, and review items are not passing, and seeded scaffolds are next actions, not evidence, until applied and re-evaluated. Full breakdown (19 expectations)
New to these terms? Start with the Quality Keeper Standards Wiki to learn what each testing type means, why it applies, and what counts as evidence. |
f90ff0c to
a96ea2c
Compare
|
Test-only: B2_REALM is optional and changes nothing when unset. Evidence is simulator only (32 pass, 5 fail, 12 skip), not real B2. One question for you: is bypassing the SSRF guard under B2_REALM acceptable, or would you prefer a loopback exception? Local typecheck/lint/test are green on current main. Update: the commit message is now a single-line subject without a body or trailer, as CONTRIBUTING.md asks. The tree is identical to before; I force-pushed (with lease) to this PR branch only, new head a96ea2c. CI on the new head: 37 success, 4 skipped, 0 failing. |
Test-only change. Lets the credential-based integration tests take an optional realm so they can also run against a B2 simulator. When
B2_REALMis unset nothing changes.liveRealmOptionsintests/helpers/live-b2.ts:{}unset, else{ realm, allowedHostSuffixes: [] }, spread into eachB2Client.B2_REALMacceptable, or prefer a loopback exception?Tracking: backblaze-labs/demand-side-ai#1556
🤖 Generated with Claude Code