Skip to content
This repository has been archived by the owner on Jun 20, 2019. It is now read-only.

Testing #1

Open
dragonwocky opened this issue Feb 24, 2017 · 19 comments
Open

Testing #1

dragonwocky opened this issue Feb 24, 2017 · 19 comments

Comments

@dragonwocky
Copy link
Member

I've set it up with Poggit..... Does that just compile it, or does it also test it and report errors?

@Thunder33345
Copy link

it only does so when you do a push or a PR is issued by someone it will do a lint check

@dragonwocky
Copy link
Member Author

Ah ok, thanks.

@SOF3
Copy link

SOF3 commented Feb 25, 2017

It doesn't test it.

Actually, I can't imagine how a plugin can be tested thoroughly automatically. Usually, an automatic run has minimal code coverage as it only detects syntax errors and errors upon startup. Not to mention there may not be enough configuration.

Anyway, Poggit does not test plugins yet. It is a planned feature in Poggit 2.0 or Poggit 3.0, which may be very distant. However, Poggit still detects syntax errors in code, as well as some common bad practice in plugins. Planning to add more with the introduction of scanning through AST.

@AvgZing
Copy link
Member

AvgZing commented Feb 25, 2017

@SOF3 it can test commands via console but not via player (as I have set up in the BoxCore Travis-CI script). If that can be implemented into poggit that would be great

@SOF3
Copy link

SOF3 commented Feb 25, 2017

Requires quite an amount of infrastructural change to support testing. Meanwhile you can try to integrate Poggit with Travis-CI by waiting for a Poggit build on Travis-CI. But, meanwhile, again, please wait for release of Poggit's API to get a build's artifact 😛

@AvgZing
Copy link
Member

AvgZing commented Feb 25, 2017

Lol okay

@catgirlinspace
Copy link
Contributor

@SOF3 Is there a travis script made to work with poggit?

@AvgZing
Copy link
Member

AvgZing commented Feb 25, 2017

@dog2puppy it's easy to create your own

@AvgZing
Copy link
Member

AvgZing commented Feb 25, 2017

No need to have a provided script.

@catgirlinspace
Copy link
Contributor

I think Travis is really hard to make a script for.

@AvgZing
Copy link
Member

AvgZing commented Feb 25, 2017

The opposite. It's easy

@catgirlinspace
Copy link
Contributor

I guess when I get home I'll try making one. The thing is, I barley know any Ternimal commands and that's what Travis uses, which makes it harder for me.

@catgirlinspace
Copy link
Contributor

Opps. Didn't mean to close.

@AvgZing
Copy link
Member

AvgZing commented Feb 25, 2017

@Dog2puppy I'll do it, if u want me to, but I really don't see the point in it. Also, see discord

@SOF3
Copy link

SOF3 commented Feb 25, 2017

It's not so simple if you want to use the particular build from Poggit, and wait for Poggit's build if Poggit has not completed it yet.

@AvgZing
Copy link
Member

AvgZing commented Feb 25, 2017

Use the *

@SOF3
Copy link

SOF3 commented Feb 25, 2017

???

@Thunder33345
Copy link

also you can install specter to send commands via player but things will get tricky

@SOF3
Copy link

SOF3 commented Feb 26, 2017

Specter does not test everything well. For example, if it's something network-level, and it doesn't test the protocol correctly...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants