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

CI and GreenKeeper #68

Open
StaticRocket opened this issue Jan 16, 2019 · 3 comments
Open

CI and GreenKeeper #68

StaticRocket opened this issue Jan 16, 2019 · 3 comments

Comments

@StaticRocket
Copy link
Collaborator

I'm looking at trying to get the /spec directory and some configs sorted out so we could potentially use Travis CI and GreenKeeper to try to limit issue sources. @Sorunome, do you think this is worth perusing? Right now (working in my fork), I've found that arduino-upload-view-spec.coffee introduces an impossible test, so I've removed. but arduino-upload-spec.coffee throws a few errors as well.

~$ apm test

FF

ArduinoUpload
  when the arduino-upload:toggle event is triggered
    it hides and shows the modal panel
      Expected '<div class="arduino-upload info-view" is="space-pen-div" style="display: none;"><button click="close" class="btn">close</button><pre></pre></div>' not to exist.
        at jasmine.Spec.<anonymous> (file:///C:/Users/Res/Desktop/arduino-upload/spec/arduino-upload-spec.coffee:19:66)
      TypeError: Cannot read property 'isVisible' of null
        at jasmine.Spec.<anonymous> (file:///C:/Users/Res/Desktop/arduino-upload/spec/arduino-upload-spec.coffee:35:31)
        at <anonymous>
    it hides and shows the view
      Expected '<div class="arduino-upload info-view" is="space-pen-div" style="display: none;"><button click="close" class="btn">close</button><pre></pre></div>' not to exist.
        at jasmine.Spec.<anonymous> (file:///C:/Users/Res/Desktop/arduino-upload/spec/arduino-upload-spec.coffee:48:66)
      Expected '<div class="arduino-upload info-view" is="space-pen-div" style="display: none;"><button click="close" class="btn">close</button><pre></pre></div>' to be visible.
        at jasmine.Spec.<anonymous> (file:///C:/Users/Res/Desktop/arduino-upload/spec/arduino-upload-spec.coffee:60:34)
        at <anonymous>


Finished in 0.215 seconds
2 tests, 7 assertions, 4 failures, 0 skipped


Tests failed
@Sorunome
Copy link
Owner

Automated tests are always a goo idea. Some of the tests seem to be wrong at first glance, as it expects e.g. the element to not exist when it actually has display:none;

@StaticRocket
Copy link
Collaborator Author

Alright, awesome. I'll poke around and see what I can figure out. I just wanted to see if this was something you'd be interested in, especially considering you ultimately control those integration settings. I'll leave this thread open in case any other questions regarding this come up.

@StaticRocket
Copy link
Collaborator Author

@Sorunome sorry for the silence. I've gotten dragged in a completely different direction by a number of things. Atom fell back on Electron4 (going against my rolling release mentality and pushing me to Neovim, a tool I had been using for quite some time now) and I've started playing with FPGAs, Assembly, Java, Machine Learning, and SysAdmin stuff to boot. I still intend on helping out wherever possible but this project is currently on the back-burner as I keep getting thrown stray projects at college.

One idea I had been working on for a revamp (to address several issues all at once) was to dump the Arduino IDE dependency and board selection stuff and just add hooks for Arduino-Makefile support. It would be a little jarring for current users but way faster and make board support and selection way cleaner. Just a thought. Best of luck with this project, dude.

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

No branches or pull requests

2 participants