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.

Sign up for GitHub

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

Jump to bottom

Add tests... #145

Open
19 tasks
ErikReider opened this issue Jul 11, 2022 · 0 comments · May be fixed by #149
Open
19 tasks

Add tests... #145

ErikReider opened this issue Jul 11, 2022 · 0 comments · May be fixed by #149
Labels
help wanted Extra attention is needed High Priority test

Comments

@ErikReider
Copy link
Owner

ErikReider commented Jul 11, 2022 •
edited
Loading

This would really speed up development due to a reduction in manually testing everything before pushing a new feature :)

Tests that I can think of. Post a comment if I missed one.

  • All swaync-client args
  • All subscription actions output a correct swaync-client subscription value
  • Blank windows being shown and not opened on the focused monitor
  • Notification actions
  • DND with different notification urgencies
  • Scripts
  • Keyboard shortcuts (if this even is possible...)
  • Styling if this even is possible (maybe screenshots with manual approval???☹️)
  • Verify notifications moving from NotificationWindow to the Control Center when cc is opened
  • Notification icon priority (provided -> app icon -> desktop entry icon -> default icon)
  • Notification icon visibility depending on config setting
  • Body markdown
  • Body image
  • CC margin and positioning
  • Notification timeouts
  • Notification visibility
  • Config and CSS expand paths (start with "~/", "/" or "./")
  • Config and CSS loading
  • Config values
@ErikReider ErikReider added help wanted Extra attention is needed High Priority test labels Jul 11, 2022
@ErikReider ErikReider linked a pull request Jul 23, 2022 that will close this issue
Draft
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted Extra attention is needed High Priority test
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Tests ErikReider/SwayNotificationCenter
1 participant
@ErikReider