Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fixing tests with stuff from master #230

Merged
merged 1 commit into from
Sep 23, 2019
Merged

fixing tests with stuff from master #230

merged 1 commit into from
Sep 23, 2019

Conversation

frodopwns
Copy link
Contributor

this fixes some things that are preventing the tests from running

@frodopwns frodopwns requested a review from jananivMS September 20, 2019 22:12
Copy link
Contributor

@jananivMS jananivMS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@frodopwns frodopwns merged commit 05b80ba into Azure:azure-sql Sep 23, 2019
Porges added a commit that referenced this pull request May 11, 2021
This reduces the number of missing-statuses from 288 to 257.

If there is an operation path like:

```
…/Microsoft.Network/dnszones/{zoneName}/{recordType}/{relativeRecordSetName}
```

And `recordType` is an enum parameter like:

```json
{
  "name": "recordType",
  "in": "path",
  "required": true,
  "type": "string",
  "description": "The type of DNS record.",
  "enum": [
    "A",
    "AAAA",
    "CNAME",
    "MX",
    "NS",
    "PTR",
    "SOA",
    "SRV",
    "TXT"
  ],
  "x-ms-enum": {
    "name": "RecordType"
  }
},
```

Then we expand the path into:

```
…/Microsoft.Network/dnszones/{zoneName}/A/{relativeRecordSetName}
…/Microsoft.Network/dnszones/{zoneName}/AAAA/{relativeRecordSetName}
…/Microsoft.Network/dnszones/{zoneName}/CNAME/{relativeRecordSetName}
…
```
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants