* Includes ts files in coverage report
* Revert "Includes ts files in coverage report"
This reverts commit 92a146247d.
* Adds ts files to coverage report
* Removes commented out code
* Revert "Removes commented out code"
This reverts commit ac7668f51f.
* Revert "Adds ts files to coverage report"
This reverts commit bb189a2236.
* Adds ts files to coverage report using remapIstanbul
* Revert "Adds ts files to coverage report using remapIstanbul"
This reverts commit 0e8885048d.
* coverage-final.json contains typescript files
* Revert "coverage-final.json contains typescript files"
This reverts commit bcfa07e332.
* Adds separate step to remap coverage files to ts.
* Configures remapIstanbul to generate a coverage report with HTML
* update remap-istanbul
* cleanup
* cleanup
Co-authored-by: Hai Cao <hacao@microsoft.com>
* Fix for paths of package.nls.*.json
* added function to generate package.nls.json file from localizedPackage.json.enu.xlf
* added generate-eng-package.nls
* removed console.log from task
* added message for build task needing to be run twice
* moved require package to inside gulp task
* remove duplicate message
* added suggested changes
* removed unused function
* added comment
* added detailed explanation comments and consolidation
* change create session test to use deferred promises
* fix test timeout
* add sqlcmd test
* add file for scripting tests
* added some node tests
* added oe utils tests
* merged master and fixed test
* fix coverage path
* test coverage
* remove out from coverage
* fix coverage site
* try new coverage path
* fix coverage report path
* add out to ignore patterns
* test new reporting style
* fix test report path
* remove html report
* try without final json
* add tests for credential store
* add tree info testing
* add more tests
* fix test
* added test for metadata
* fix tests
* add command to focus on results grid
* show connection info in results view as well
* added context menu option for just copying headers
* fixed bug where moving to text editor and refreshibng would show resutls twice
* added use statement for OE query context menu actions
* show xml in results pane
* re-enable code coverage on travis CI
* run tasks only for master branch
* webview apis
* got it working
* working with no compile errors
* data not coming yet
* data coming in
* fix styling
* fixed theme styling
* cleaned code
* removed unnecessary files
* fix context menu positioning
* fixed folder structure and compilation
* removed unnecessary refresh mechanism
* added content security policy tags
* added copyright headers
* fixed closing of webviews
* added fixed theming without explicit css, need refresh
* fixed tab switching
* fix bug where changing focus would re-run
* reuse query controller for queries in the panel
* fixed theming changes
* fix conext menu position using angular instead of jquery
* fixed batch linking from resultset
* access denied, need to add to local resource uri
* grid showing without results
* make into one project, needs cleaning
* remove changes for angular
* fix rxjs version
* fixed query runner tests
* fix sqloutput tests
* fixed cancel query
* added webviewpanel controller tests
* added OE to new proj
* fixed all tests
* remove borwser tests from the pipeline
* fix html order
* initial code review comments
* removed commented code
* fix packaged builds
* fix angular package version
* fix the test reports location
* Clean up linting errors and add in SDL-required rules.
* Remove unnecessary line
* Add linting for files under js folder - there are typescript files there that should be included.
* Add naming check rules
* Use correct tag for gulp package
For new machine setup, running
pm install errors because the tag used for the gulp package does not exist (4.0), it should be (v4.0.0).
* Don't use typings of the parent folder
* VBump for GA Release
- Fixes#828
- Fixes#829
Updated package version, added release notes and changelog data. Note: did not update SqlToolsService as we're working to get a 1.0 build with all bits uploaded to GitHub. That will be done in a separate PR.
Fixes#385. Added keyboard shortcut Ctrl+C and context menu support for copying from the Messages section. To do this:
Messages have a context menu handler with a single option of Copy.
Added use of the Rangy library which simplifies selection and text range manipulation, including pretty print support for copying as text. We need to copy as text since the HTML includes our anchor links to the current page and in dark mode the text is copied as black text on a black background
- Copy works with validation that only messages are copied
- Using Rangy's text() function to get correctly formatted text. This ensures it is tab delimited which is copiable into Excel and text editors in the expected format
- Copy shortcut key supports copying Messages
* added basic selection tests
* added coverage based on config; added coverage for js plugins
* updated which coverage file was being used
* added check for config file
* fix karma errors
* adding tests
* karma setup
* got tests working
* mocking working
* set up coverage
* cleaned up commit
* moved tasks to appropriate places
* added task for jenkins
* change jenkins to cproc
* testing jenkins build
* testing builds
* combined testing
* finished build error
* fixed testing reporting
* added tests for coverage up to ~70%
* up to 85% coverage
* finished testing and clean up karma setup
* reverted unintended changes
* moved tests; added extraction function
- Added Cobertura reporting and moved logic to the coverconfig.json so it can easily be changed on demand
- Clean up the coverage folder on each coverage run, and no longer create a new json file for each run
- Included a new launch config that supports running with code coverage.
- Coverage is stored in a coverage folder as both LCOV file and HTML report (in sub folder)
- Added new scripts in our build machine to run with code coverage. Will enable a run that uses code coverage once this is merged
- To view code coverage while developing, install the LCOV VSCode extension, enable it and then hit F1 -> LCOV Menu -> Enable Decorations. This will highlight covered / missed lines in green and red.
Note: The HTMLContent codebase is not currently covered. With recent changes to minify this it breaks the remapping tool we use.
Detailed notes
- Coverage uses a config file to load settings. This is copied at the start of each task
- If tests fail, the process exit code will be 1. We should still write out code coverage in these cases as this is a useful detail even on runs with some test failures.
- On Windows, the extension is loaded (pulling in all source files via require) before the test runner is configured. Since instrumentation happens by hooking the require call, this meant that instrumentation never happened on Windows. To resolve this, we invalidate the Module cache for our source files so they are reloaded the next time anything requires them. At this point they are instrumented as expected. There is a minor risk to this if we had some global state or deeper integration test, but this doesn't impact any existing unit tests and is needed to unblock code coverage
* Skip coverage of HTMLContent due to crash
- With the latest HTMLContent changes to minify & uglify the code, the remap-istanbul component responsible for js -> ts remapping crashes trying to parse the sourceMap entries in the file. This is due to lack of whitespace between code which it expects - basically, it's not set up to run against such compact code
- Disabling coverage of this part of the code until this issue is resolved
Alex Ma
Alan Ren
Charles Gagnon
Cheena Malhotra
Lewis Sanchez
Aasim Khan
Aditya Bist
Emad Alashi
Matt Irvine
Leila Lali
Anthony Dresser
Kevin Cunnane
Raymond Martin
Anthony Dresser
Karl Burtram