Граф коммитов

81 Коммитов

Автор SHA1 Сообщение Дата
Ravi Chande 8262fcdcf5 Make test less strict 2018-10-04 13:34:59 -07:00
Akshita Agarwal 5a16cdd7dc Make tests pass 2018-08-28 12:22:26 -07:00
Akshita Agarwal a792284d85 Use completion 2018-08-28 12:21:44 -07:00
Akshita Agarwal 94ad2be4aa Use completion class 2018-08-28 12:21:44 -07:00
Akshita Agarwal e5d3b43201 Change package json
Add completion integration test

changes

Remove main function

Add tests for the completion provider
2018-08-28 12:20:59 -07:00
Akshita Agarwal 5f5a2f1697 Use the latest omnisharp 2018-08-16 17:10:44 -07:00
Akshita Agarwal 51da252e07
Merge branch 'master' into nimullen/2431 2018-08-01 11:01:49 -07:00
Akshita Agarwal d1d8453fac check if normal omnisharp passes test
Use an old version

use another version

Yet another omnisharp version

using next version of yet another version
2018-07-31 17:14:54 -07:00
N. Taylor Mullen 96f578cb54 Addressed code review comments.
- Moved bits to async await
- Added integration test
- Used event stream APIs
2018-07-26 15:56:15 -07:00
Akshita Agarwal 98b6afc3c1 Use the integration helpers 2018-07-19 12:23:55 -07:00
Akshita Agarwal f44c31d029 Test for the implementation provider 2018-07-19 12:23:55 -07:00
Dustin Campbell 2d4907e261 Improve integration test stability and add tests for code lens options 2018-07-09 14:58:04 -07:00
Dustin Campbell d90593ceab Refactor a bit and add some more test messages 2018-07-09 13:37:19 -07:00
Dustin Campbell 9507516fb3 Restore test assets at suite setup 2018-07-09 13:37:19 -07:00
Dustin Campbell 72986b750a Remove all 'bin' and 'obj' files from test assets 2018-07-09 13:37:18 -07:00
Dustin Campbell 0367c560c1 Tweak some assertions 2018-07-09 13:35:22 -07:00
Dustin Campbell bf6edd0346 Move suiteTeardown() methods into the actual suites 2018-07-09 13:35:22 -07:00
Dustin Campbell df75c24cb5 Add back call to clean up workspace in launch configuration test 2018-07-09 13:35:22 -07:00
Dustin Campbell eba39a7e0d Add codeLensProvider integration test 2018-07-09 13:35:22 -07:00
Dustin Campbell 1d19fdb118 Add documentSymbolProvider integration test 2018-07-09 13:35:21 -07:00
Dustin Campbell 620fdf4364 Unify how integration tests activate the C# extension 2018-07-09 13:35:21 -07:00
Janaka Abeywardhana 1e0e8fad62 Fix: test project missing lib ref in csproj file 2018-07-04 22:16:19 +01:00
Janaka Abeywardhana c80dd09586 Fix: missing lib ref in test projet sln files 2018-07-04 22:13:05 +01:00
Janaka Abeywardhana f801b8e4e6 Merge: latest upstream master 2018-07-02 20:56:51 +01:00
Ravi Chande 64b981d039 Fix typo in settings.json 2018-06-27 09:42:11 -07:00
Ravi Chande 906d76b06e Cleanup/simplify 2018-06-14 09:08:15 -07:00
Janaka A c2eea4af2b Merge: latest upstream/master 2018-06-14 00:59:47 +01:00
Akshita Agarwal 5669d43a55 Add type info in integration tests 2018-05-18 15:56:05 -07:00
Janaka Abeywardhana fe33a07d27 Merge in latest upstream/master 2018-05-15 22:02:32 +01:00
Ravi Chande e1ec83a0ef move back to latest for O# release 2018-04-25 15:40:04 -07:00
akshita31 c218dfaa8a
Divide the package manager into separate components (#2188)
* Feature tests running with refactored package manager

* Refactoring packages-1

* Test for the downloader running using the mock server

* Using network settings

* Changing the package path

* Dividing packages.ts into separate components

* use filter async

* Use tmpfile interface

* Check for the event stream

* Using FilePathResolver

* Remove using

* Testing for normal and fallback case working

* Resolve the paths

* Remove failing case

* package installer test-1

* Add package installer test

* Create tmp asset

* Package Downloader test refactored

* Rename files

* resolve compile issues

* Clean up installer

* Clean up the tests

* Rename packages

* Package installer test

* PR feedback

* Package Filter test

* Remove yauzl.d.ts

* Filter test

* Added test for invalid zip file

* method for getting the request options

* remove imports

* please resolve the path

* remove  latest in settings

* Package Manager test executing

* Use free port in package manager test

* Package Manager (using a https server running)

* using http mock server

* Downloader test running using free port
2018-04-24 14:29:51 -07:00
Piotr Puszkiewicz 32109f55aa
unify rx usage to rxjs (#2168)
The current codebase uses both rx [v4] and rxjs [v5] implementations. This PR consolidates our use of rx onto the v5 libraries.
2018-04-12 13:59:23 -07:00
Piotr Puszkiewicz 563a1aa58d
tsconfig.json: noUnusedLocals, noFallThroughCaseInSwitch, tslint.json: promise-function-async (#2162)
Adds noUnusedLocals to tsconfig.json to keep our sources clean
Adds noFallThroughCaseInSwitch in tsconfig.json to prevent unintended switch behavior
Adds promise-function-async to tslint.json to force all async functions to be marked as async. This is a building block towards eliminating promises in favor of async/await.
2018-04-09 09:30:10 -07:00
Janaka A 1aa5259f9b Fix: based on comments in PR #2053
- remove isDefualt property from target interface
- move is-default logic from resourcesToLaunchTargets() to autostart()
- refactor autostart() target logic and comments
- rename test .NET solution files
2018-04-07 18:46:40 +01:00
Janaka A 7d2b1ab8b1 Merge upstream master 2018-04-07 14:30:52 +01:00
Piotr Puszkiewicz 1f3e77a04b
Enable `noImplicitAny` and `alwaysStrict` in `tsconfig.json (#2159)
Along the way, fixed a few bugs that were in place due to implicit anys. Also removed dependency on deprecated vscode API.
2018-04-07 01:33:28 -07:00
Piotr Puszkiewicz ed60379138
Test the vsix, not the build layout (#2156)
The VSCode C# Extension build process follows the VS Code docs and runs tests directly inside of its repo root. This unfortunately gives a false sense of security because bugs can be introduced during VSIX packaging [particularly due to missing content in node_modules or excluded via .vscodeignore].

This change addresses this problem by moving our CI tests to execute the VSIX instead of the build's intermediate artifacts. Specifically:

build the vsix
unpackage the vsix
instrument the unpackaged vsix
run tests with VS Code Host pointing to the unpackaged vsix
This makes our CI tests ~= to the user's runtime experience and will greatly help us with size reduction efforts.

To support this change, I also moved our build system from package.json to Gulp. This makes the build scripts significantly easier to understand, provides intellisense for build scripts, and build-time type checking for their contents.

I also strengthened the repo's use of .vscodeignore by creating a copy of the file for each scenario [online packages and offline packages]. The new gulp packaging scripts take advantage of these files to produce packages with predictable contents regardless of when packaging occurs. [small caveat, @akshita31 will be adding a test that validates that net-new content does not start sneaking into the vsix package].
2018-04-06 14:17:17 -07:00
akshita31 da3248956e
Clean up signature help test (#2117)
* Added parameter

* modifications

* Cleaned up tests
2018-03-19 13:05:50 -07:00
Ravi Chande 27aab37cf2 Refactor logger, reporter and channel into a unified EventStream using Rx Observable (#2084)
* Changes to refactor logging in server

* Adding packages

* Changes

* Remove reporter from CSharpExtDownloader

* remove telemtery reporter from server.ts

* remove reporter from definitionProvider

* Remove reporter from dotnetTest.ts

* Debugger Activation + Commands

* reduce message types

* remove reporter from commands.ts

* remove channel from  status.ts

* Remove reporter & logger from extension.ts

* Build issues

* Add missing rx dependency

* Changed to download progress

* Removed using and pass platformInfo

* Moved files in observer folder

* Renamed the files and added omnisharp channel observer

* Remove unnecessary format

* Changes in main.ts

* Remove channel from global declaration

* Preserving the context in onNext invocations

* Pulled platformInfo out of server

* Remove unnecessary variable

* Formatting

* Renamed observers

* Add mocha+wallaby tests

eventually the feature tests should be removed and most of our tests should become unit tests that are runnable from the command line or via wallaby.

npm run tdd will enable using mocha's command line tdd capability

* Code clean up

* Fix `tdd` command

* Fix test paths

* Add initial DotnetChannelObserver test

* Testing the download messages

* Remove logger from requestQueue.ts

* Fix builds

* Use package manager factory

* Remove Lines

* Remove extra appendLine

* Added test for csharp logger and channel

* Extracted base class for observers

* Test having dependency on vscode

* vscode adapter changes

* Changes for adapter

* Refactored Omnisharp Manager

* Moved from interfaces to classes

* Renamed onNext to post

* Created class EventStream

* Removed comment

* Added missing break

* Added test for Omnisharp Logger

* Test for OmnisharpLoggerObserver

* Test for telemetry reporter observer

* Added test for all the observers

* minor nits

* Changes

* Remove unnecessary imports

* remove import

* Modified failing test

* Make tests pass

* Renamed platformInfo

* CR feedback
2018-03-14 14:19:01 -07:00
Piotr Puszkiewicz 007bc0e985
Add product-wide code coverage + codecov.io integration (#2101)
Add product-wide code coverage + codecov.io integration

Several new scripts were added:

npm run cov:instrument: rebuilds your sources, then instruments them for coverage. Subsequent 

npm run test will generate coverage data into the .nyc_output directory

npm run cov:merge-html: merges all reports from .nyc_output and puts a locally viewable coverage report into coverage directory
2018-03-12 14:14:00 -07:00
Akshita Agarwal 694b8d78be Added settings file 2018-02-28 10:36:01 -08:00
Akshita Agarwal 180f4385d3 Renamed experimentalId and some methods 2018-02-23 17:12:42 -08:00
akshita31 7f6285535e
Using Structured Documentation for Signature Help and improving Parameter Documentation (#1958)
* Structured Documentation in Signature Help

* Code clean up

*  Using only summary of the documentation

* Code clean up

* Documentation for parameters showing in signature help

* Removing unnecesaary import

* Using interploated string and fixed spacing

* Parameter Documentation using interpolated text

* Added tests
2018-02-21 15:36:32 -08:00
akshita31 2bb822db3f
Make hover test run by adding the file in the project (#2058)
* Added hover.cs file

* Not requiring server restart
2018-02-20 18:23:51 -08:00
Janaka A 86b1b72f58 Refactor: `defaultLaunchTarget` to `defaultLaunchSolution` 2018-02-18 20:02:01 +00:00
Janaka A 8a69508607 remove none existant project from test sln 2018-02-18 19:37:42 +00:00
Janaka A 290ff42fda Fix: gitignore 2018-02-18 18:43:28 +00:00
Janaka A 5c7d70ac12 Merge: latest origin master 2018-02-18 18:40:51 +00:00
Janaka A 2a5056ac95 tweak gitignore 2018-02-18 18:30:28 +00:00
Janaka A baa827b5d7 Add: gitignores to test projects 2018-02-18 18:21:27 +00:00