Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add integration tests for plugin system #342

Open
Hornwitser opened this issue Dec 6, 2020 · 0 comments
Open

Add integration tests for plugin system #342

Hornwitser opened this issue Dec 6, 2020 · 0 comments
Labels
Good First Issue Feature / Fix does not require in-depth knowlege of Clusterio Release: 2.0 Required before release of Clusterio 2.0

Comments

@Hornwitser
Copy link
Member

Hornwitser commented Dec 6, 2020

There are no tests currently covering if plugins are loaded, this task consists of the following items:

  • Create a test plugin in test/files
  • Add it to be loaded in test/integration/index.js
  • Add a clusterctl command to it and test that it's present in test/integration/clusterio.js
  • Add a control-controller request and test that it works.
  • Add a controller-host event and test that BaseControlPlugin.broadcastEventToHosts work.
  • Add a control-host request and test that it works.
  • Add a control-instance request and test that it works on a running instance.
  • Add a Lua module to the plugin and use a request like the previous entry to test that it is present with an rcon command.
  • Add a request producing log statements in the plugin at the controller, host and instances level and verify log messages are correctly tagged with the plugin and host/instance id for the host/instance plugin code.
@Hornwitser Hornwitser added Release: 2.0 Required before release of Clusterio 2.0 Good First Issue Feature / Fix does not require in-depth knowlege of Clusterio labels Dec 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Good First Issue Feature / Fix does not require in-depth knowlege of Clusterio Release: 2.0 Required before release of Clusterio 2.0
Projects
None yet
Development

No branches or pull requests

1 participant