Skip to content

First version of cache builder and lookup logic for live validation#93

Merged
amarzavery merged 10 commits into
Azure:masterfrom
vishrutshah:init-cache
Apr 13, 2017
Merged

First version of cache builder and lookup logic for live validation#93
amarzavery merged 10 commits into
Azure:masterfrom
vishrutshah:init-cache

Conversation

@vishrutshah

@vishrutshah vishrutshah commented Apr 11, 2017

Copy link
Copy Markdown
Contributor
  • Clones git repository from configurations
  • Read all the swagger spec files and parse them
  • Create an in memory cache for all the operations across all the swagger specs based on below structure:
{
  "api-version1":
  {
    "provider1": 
     {
        "get": [ {opeationObject(Sway)}, {operationObject(sway)}, . .  ],
        "post": [ {opeationObject(Sway)}, {operationObject(sway)}, . .  ],
        "put": [ {opeationObject(Sway)}, {operationObject(sway)}, . .  ],
        ...
     }
  },
  "api-version2":
  {
    "provider2": 
     {
        "delete": [ {opeationObject(Sway)}, {operationObject(sway)}, . .  ],
        "head": [ {opeationObject(Sway)}, {operationObject(sway)}, . .  ],
        "get": [ {opeationObject(Sway)}, {operationObject(sway)}, . .  ],
        ...
     }
  },
  . . .
}
  • Expose getPotentialOperations for lookup for best match on regex of path

Part of #89 & #90

@msftclas

Copy link
Copy Markdown

@vishrutshah,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@vishrutshah

Copy link
Copy Markdown
Contributor Author

Closing and reopening to run travis

@vishrutshah vishrutshah reopened this Apr 13, 2017
@msftclas

Copy link
Copy Markdown

@vishrutshah,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@amarzavery

Copy link
Copy Markdown
Contributor

@vishrutshah શું એક અદ્ભુત કામ ડ્યૂડ. તમે પ્રથમ પુલ વિનંતી ટ્રેવિસ CI સક્રિય થયેલ

Comment thread test/liveValidatorTests.js Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

indentation 2 spaces

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

sure i'll update my vscode config for 2 space indentations 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants