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

[testing] Research test framework #2

Closed
svenefftinge opened this issue Mar 1, 2017 · 1 comment
Closed

[testing] Research test framework #2

svenefftinge opened this issue Mar 1, 2017 · 1 comment

Comments

@svenefftinge
Copy link
Contributor

For writing unit tests we should decide for a testing framework to use.
My requirements would be:

  • short turnarounds / incremental test runs
  • readability / little boiler-plate
  • nice: native support in webstorm for debugging
@svenefftinge
Copy link
Contributor Author

We agreed we are going with mocha.

marcdumais-work referenced this issue in marcdumais-work/theia Jun 1, 2017
Merge pull request #2 from marcdumais-work/gdb-console
1d306d3 
[debug] mode backend code to the node folder
@MarkZ3 MarkZ3 mentioned this issue Dec 18, 2017
Closed
@epatpol epatpol mentioned this issue Jan 18, 2018
Closed
@kittaakos kittaakos mentioned this issue Jan 30, 2018
Closed
lmcbout added a commit that referenced this issue Mar 14, 2018
@lmcbout
# This is a combination of 2 commits.
8c32c86 
# This is the 1st commit message:

GH-1081 Auto-dismiss notification.
Add notifications preferences to allow auto-dismiss
Auto-Dismiss can be used/disabled for each notification since it is optional
When the user has more than one action available,
the auto-Dismiss timeout will not be applied

Signed-off-by: Jacques Bouthillier <jacques.bouthillier@ericsson.com>

# This is the commit message #2:

fix issue with `timeout=0`

Signed-off-by: Alex Tugarev <alex.tugarev@typefox.io>
lmcbout added a commit that referenced this issue Mar 14, 2018
@lmcbout
GH-1081 Auto-dismiss notification.
1a2ea25 
Add notifications preferences to allow auto-dismiss
Auto-Dismiss can be used/disabled for each notification since it is optional
When the user has more than one action available,
the auto-Dismiss timeout will not be applied

Signed-off-by: Jacques Bouthillier <jacques.bouthillier@ericsson.com>

# This is the commit message #2:

fix issue with `timeout=0`

Signed-off-by: Alex Tugarev <alex.tugarev@typefox.io>
lmcbout added a commit that referenced this issue Mar 14, 2018
@lmcbout
GH-1081 Auto-dismiss notification.
0f6ff70 
Add notifications preferences to allow auto-dismiss
Auto-Dismiss can be used/disabled for each notification since it is optional
When the user has more than one action available,
the auto-Dismiss timeout will not be applied

Signed-off-by: Jacques Bouthillier <jacques.bouthillier@ericsson.com>

# This is the commit message #2:

fix issue with `timeout=0`

Signed-off-by: Alex Tugarev <alex.tugarev@typefox.io>
@elaihau elaihau mentioned this issue Mar 28, 2018
Closed
benoitf pushed a commit that referenced this issue Jun 20, 2018
@olexii4 @benoitf
Add notification manager API (#2)
03fbddd 
Signed-off-by: Oleksii Orel <oorel@redhat.com>
bhufmann pushed a commit to bhufmann/theia that referenced this issue Jul 26, 2018
@lmcbout @bhufmann
eclipse-theiaGH-1081 Auto-dismiss notification.
cd7ec79 
Add notifications preferences to allow auto-dismiss
Auto-Dismiss can be used/disabled for each notification since it is optional
When the user has more than one action available,
the auto-Dismiss timeout will not be applied

Signed-off-by: Jacques Bouthillier <jacques.bouthillier@ericsson.com>

# This is the commit message eclipse-theia#2:

fix issue with `timeout=0`

Signed-off-by: Alex Tugarev <alex.tugarev@typefox.io>
@elaihau elaihau mentioned this issue Jan 28, 2019
Merged
@vince-fugnitto vince-fugnitto mentioned this issue Jul 23, 2019
Closed
@lmcbout lmcbout mentioned this issue Mar 16, 2020
Merged
1 task
@paul-marechal paul-marechal mentioned this issue May 5, 2020
Closed
@danarad05 danarad05 mentioned this issue Oct 8, 2020
Closed
@RomanNikitenko RomanNikitenko mentioned this issue Oct 26, 2020
Merged
1 task
ndoschek added a commit to ndoschek/theia that referenced this issue Jan 20, 2021
@ndoschek
Address review comments eclipse-theia#2
281ed4a 
Signed-off-by: Nina Doschek <ndoschek@eclipsesource.com>
ndoschek added a commit to ndoschek/theia that referenced this issue Jan 21, 2021
@ndoschek
Address review comments eclipse-theia#2
7d70f30 
Signed-off-by: Nina Doschek <ndoschek@eclipsesource.com>
@marcdumais-work marcdumais-work mentioned this issue Feb 24, 2021
Open
@tsmaeder tsmaeder mentioned this issue Aug 9, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@svenefftinge @kittaakos