GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Make it easy to test plugins, and Implement test suite for them. · Issue #481 · feather-rs/feather

Make it easy to test plugins, and Implement test suite for them. #481

New issue
Copy link
New issue
Copy link
Open
Open
Make it easy to test plugins, and Implement test suite for them.#481
Copy link

Description

@Miro-Andrin
Miro-Andrin
opened on Sep 24, 2021
Issue body actions

Context

Currently there is no automated/easy way to test quill plugins. Making it hard to detect regressions.
We need a way of easily compile multiple plugins and run them on a server. We also need a way
for plugins to signal if something bad has happened.

I believe there are many layers to this issue, and that a good approach would be to split it up into
some smaller improvements, that in them selves probably are a good idea, but that as a whole
gives us the desired result.

Here is what i suggest. If you believe this is wrong, and a better approach exists, then do tell.
Is there a different route to a similar solution, or maybe some of this is already partly implemented?

Improve cargo-quill

I suggest adding a command to cargo quill that given a feather server (binary or source code directory) , and one or more plugins (either as a .plugin file or source code). Does the necessary compiling, moves the plugins into the correct spot, and starts the server. The command should support a flag for specifying if a native or wasm plugin should be generated.

Add sport for plugins to stop.

Add an api to plugins for them to signal that they are done . Basically a way for plugins to unload themselves,
with a success or error code. Maybe even a short plugin message could be printed to the console.

Add a test mode

Finally we add a flag like --test-plugins to the server binary. If this flag is set, then the server starts every plugin,
and runs them until all plugins have unloaded themselves. Finally printing what plugins ended in an error state.

With all of these steps combined it should be straight forward to create a script to automatically test plugins before
merges.

Activity

  1. ambeeeeee commented on Sep 25, 2021

    @ambeeeeee
    ambeeeeee
    on Sep 25, 2021
    Contributor
    More actions

    (the main reason I wanted to revert was to fix code quality issues that just pushed me to actually do it instead of doing literally anything else like I had been)

    I still agree with the idea!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions

      You are visitor 01402503 — GeezerVison does not store the pages you look up.
      Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.