Added support for test automation

StefanMaron / BusinessCentral.LinterCop

Community driven code linter for AL (MS Dynamics 365 Business Central)

https://stefanmaron.com

MIT License

68 stars 27 forks source link

Added support for test automation #648

Closed christophstuber closed 1 week ago

christophstuber commented 1 month ago

New NUnit test project BusinessCentral.LinterCop.Test and new folder structure.
Created a fork of RoslynTestKit by cezarypiatek with support for AL language.
Added first test cases for rules 0001 to 0019.

Arthurvdv commented 1 month ago

This looks really awesome, thank you for contributing this!

We’ve lacked the possibility to have automated testing so far, this is of great help for the project.

Due to the BCTechDays in Antwerp this week, I’m a bit short on time to check this out right now, but absolutely next week I’m going to have a try on this.

Arthurvdv commented 1 week ago

Hi @christophstuber, I'm struggling to resolve these conflicts and add them to this PR somehow. Do you mind to do a merge from the pre-release branch into this PR?

christophstuber commented 1 week ago

@Arthurvdv Conflicts should be resolved now. I also added some more tests up to rule 0032.

@pri-kise Also added a new section to the readme.

Arthurvdv commented 1 week ago

@christophstuber Awesome, and again thank you for contributing!

dannoe commented 5 days ago

Hi @christophstuber, thanks for contributing the automated tests. I cloned the prerelease branch and gave it a try. I had to manually copy the Microsoft.Dynamics.Nav*dlls because they are set to private=true in the csproj file. Was this intentional, and if so, can you explain why? If not, I am more than willing to add a pull request to change this.