Error: Invalid reference token: /Tables(\'{tableName}\')
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 35/100
Research direction
Reproduce the warning using the Cosmos DB Tables conversion and the linked GitHub Actions run. Start with @azure/oad/dist/lib/validators/openApiDiff.js, then trace the calls from eng/tools/openapi-diff-runner/dist/src/run-oad.js and detect-breaking-change.js. Determine whether the invalid reference token is harmless or indicates missed validation, and verify the chosen behavior against the same-version breaking-change check.
Written by the indexing model from the issue text.
Description
In this PR, converting tables to TSP:
openapi-diff prints a lot of messages like this. They are just logged as warnings, so maybe they're harmless, but they might also be a symptom of a larger problem (like missing validation).
ENTER definition runOad oldSpec: /home/runner/work/azure-rest-api-specs/same-version-c93b354fd9c14905bb574a8834c4d69b/specification/cosmos-db/data-plane/Tables/stable/2019-02-02/table.json, newSpec: specification/cosmos-db/data-plane/Tables/stable/2019-02-02/table.json, oldTag: undefined, newTag: undefined
oad.CompareTags() when !(oldTag && newTag)
Error: Invalid reference token: /Tables(\'{tableName}\')
at api.get (/home/runner/work/azure-rest-api-specs/azure-rest-api-specs/node_modules/json-pointer/index.js:54:19)
at updateChangeProperties (/home/runner/work/azure-rest-api-specs/azure-rest-api-specs/node_modules/@azure/oad/dist/lib/validators/openApiDiff.js:30:58)
at /home/runner/work/azure-rest-api-specs/azure-rest-api-specs/node_modules/@azure/oad/dist/lib/validators/openApiDiff.js:247:105
at Array.map (<anonymous>)
at OpenApiDiff.processViaOpenApiDiff (/home/runner/work/azure-rest-api-specs/azure-rest-api-specs/node_modules/@azure/oad/dist/lib/validators/openApiDiff.js:247:40)
at async runOad (file:///home/runner/work/azure-rest-api-specs/azure-rest-api-specs/eng/tools/openapi-diff-runner/dist/src/run-oad.js:48:28)
at async doBreakingChangeDetection (file:///home/runner/work/azure-rest-api-specs/azure-rest-api-specs/eng/tools/openapi-diff-runner/dist/src/detect-breaking-change.js:188:29)
at async checkBreakingChangeOnSameVersion (file:///home/runner/work/azure-rest-api-specs/azure-rest-api-specs/eng/tools/openapi-diff-runner/dist/src/detect-breaking-change.js:71:48)
at async validateBreakingChange (file:///home/runner/work/azure-rest-api-specs/azure-rest-api-specs/eng/tools/openapi-diff-runner/dist/src/commands.js:107:17)
at async main (file:///home/runner/work/azure-rest-api-specs/azure-rest-api-specs/eng/tools/openapi-diff-runner/dist/src/index.js:152:18)
- Dominant language
- C#
- Stars
- 290
- Forks
- 50
- Avg merge
- 2d 21h
- Merged PRs (30d)
- 10
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from Azure/openapi-diff
-
Azure/openapi-diff#482 · 1 assignee ·
-
Azure/openapi-diff#479 · 2 comments · 1 assignee ·
-
Azure/openapi-diff#456 · 1 assignee ·
-
Azure/openapi-diff#371 · 7 comments · 1 assignee ·
-
Difficulty 4/5 3-5 days Newbie friendliness 25/100
Azure/openapi-diff#364 · 4 comments ·
All issues in Azure/openapi-diff
Similar issues
-
Documentation
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
cake-build/cake#5024 ·
-
Frontend status/draft TechnicalDebt
Difficulty 2/5 1-2 days Newbie friendliness 75/100
Altinn/altinn-auth#4143 ·
-
.NET Flaky Test Testing Tests
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
getsentry/sentry-dotnet#5617 · 1 comment ·
-
:watch: Not Triaged dotnet-fsharp/svc
Difficulty 1/5 Under an hour Newbie friendliness 90/100
-
Client customer-reported needs-team-attention question Service Attention WebPubSub
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
Azure/azure-sdk-for-net#63292 · 3 comments · 1 reaction ·