- corrections at appveyor.yml config - Githubissues

oleg-shilo / cs-script.npp

CS-Script (C# Intellisense) plugin for Notepad++ (x86/x64)

MIT License

246 stars 52 forks source link

- corrections at appveyor.yml config #8

Closed chcg closed 6 years ago

chcg commented 6 years ago

use v141 for deployment tests

, see https://ci.appveyor.com/project/chcg/cs-script-npp/build/1.7.6.24

Seems another file (NppShortcutRemapper.exe) is missing:

PreBuildEvent: copy "C:\projects\cs-script-npp\src\CSScriptIntellisense..\NppShortcutRemapper\bin\NppShortcutRemapper.exe" "C:\projects\cs-script-npp\src\CSScriptIntellisense\Resources\NppShortcutRemapper.exe"

xunit tests are running, see https://ci.appveyor.com/project/chcg/cs-script-npp/build/1.7.6.24/job/vxc4s2l2t8pyda7x/tests , but most of them are failing, because RoslynIntellisense.exe could not be found as it is searched for like in the N++ plugin installation configuration by CSScriptIntellisense.RoslynCompletionEngine.Init()
```
        var file = Roslyn.LocateInPluginDir("RoslynIntellisense.exe", "Roslyn", @".\");
```

Filepath is something like: "C:\Users\XXX\AppData\Local\Temp\99a134b2-8e5f-4ca6-ae02-05b468e30091\99a134b2-8e5f-4ca6-ae02-05b468e30091\assembly\dl3\ef37f93b\c0660d5c_5a5ed301\RoslynIntellisense.exe"

Is it necessary to set roslynintellisense_path for running the unittests?

oleg-shilo commented 6 years ago

Yeah, I haven't updated the tests to reflect the current implementation as the big rework is coming. I will be replacing RoslynIntellisense.dll with its portable version syntaxer.exe.

But I will review and fix/disable the offending test asap as I am about to start the syntaxer.exe integration.

chcg commented 6 years ago

Ok, so I will wait until this major rework is lifted and then check again what is left over to get a proper appveyor build run

oleg-shilo commented 6 years ago

The latest commit should fix the problems. I have updated some tests and disabled the ones that will be changed as the result of Syntaxer migration.