Skip to content

Commit f11657a

Browse files
authored
e2e fixes + CONTRIBUTING.md (#435)
* fixing tests * adding info to CONTRIBUTING.md
1 parent 285bdbc commit f11657a

File tree

4 files changed

+5
-24
lines changed

4 files changed

+5
-24
lines changed

CONTRIBUTING.md

+3-22
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,8 @@ Working on your first Pull Request? You can learn how from this *free* series, [
1717

1818
## Submitting code
1919

20-
Any code change should be submitted as a pull request. The description should explain what the code does and give steps to execute it. The pull request should also contain tests.
20+
Any code change should be submitted as a pull request to the "next" branch. The description should explain what the code does and give steps to execute it. The pull request should also contain tests (unit/e2e).
21+
Also, before submitting a PR please make sure to open an issue for it.
2122

2223
## Code review process
2324

@@ -34,8 +35,6 @@ Anyone can file an expense. If the expense makes sense for the development of th
3435
If you have any questions, create an [issue](issue) (protip: do a quick search first to see if someone else didn't ask the same question before!).
3536
You can also reach us at [email protected].
3637

37-
## Credits
38-
3938
### Contributors
4039

4140
Thank you to all the people who have already contributed to angular-datepicker!
@@ -46,22 +45,4 @@ Thank you to all the people who have already contributed to angular-datepicker!
4645

4746
Thank you to all our backers! [[Become a backer](https://opencollective.com/angular-datepicker#backer)]
4847

49-
<a href="https://opencollective.com/angular-datepicker#backers" target="_blank"><img src="https://opencollective.com/angular-datepicker/backers.svg?width=890"></a>
50-
51-
52-
### Sponsors
53-
54-
Thank you to all our sponsors! (please ask your company to also support this open source project by [becoming a sponsor](https://opencollective.com/angular-datepicker#sponsor))
55-
56-
<a href="https://opencollective.com/angular-datepicker/sponsor/0/website" target="_blank"><img src="https://opencollective.com/angular-datepicker/sponsor/0/avatar.svg"></a>
57-
<a href="https://opencollective.com/angular-datepicker/sponsor/1/website" target="_blank"><img src="https://opencollective.com/angular-datepicker/sponsor/1/avatar.svg"></a>
58-
<a href="https://opencollective.com/angular-datepicker/sponsor/2/website" target="_blank"><img src="https://opencollective.com/angular-datepicker/sponsor/2/avatar.svg"></a>
59-
<a href="https://opencollective.com/angular-datepicker/sponsor/3/website" target="_blank"><img src="https://opencollective.com/angular-datepicker/sponsor/3/avatar.svg"></a>
60-
<a href="https://opencollective.com/angular-datepicker/sponsor/4/website" target="_blank"><img src="https://opencollective.com/angular-datepicker/sponsor/4/avatar.svg"></a>
61-
<a href="https://opencollective.com/angular-datepicker/sponsor/5/website" target="_blank"><img src="https://opencollective.com/angular-datepicker/sponsor/5/avatar.svg"></a>
62-
<a href="https://opencollective.com/angular-datepicker/sponsor/6/website" target="_blank"><img src="https://opencollective.com/angular-datepicker/sponsor/6/avatar.svg"></a>
63-
<a href="https://opencollective.com/angular-datepicker/sponsor/7/website" target="_blank"><img src="https://opencollective.com/angular-datepicker/sponsor/7/avatar.svg"></a>
64-
<a href="https://opencollective.com/angular-datepicker/sponsor/8/website" target="_blank"><img src="https://opencollective.com/angular-datepicker/sponsor/8/avatar.svg"></a>
65-
<a href="https://opencollective.com/angular-datepicker/sponsor/9/website" target="_blank"><img src="https://opencollective.com/angular-datepicker/sponsor/9/avatar.svg"></a>
66-
67-
<!-- This `CONTRIBUTING.md` is based on @nayafia's template https://github.com/nayafia/contributing-template -->
48+
<a href="https://opencollective.com/angular-datepicker#backers" target="_blank"><img src="https://opencollective.com/angular-datepicker/backers.svg?width=890"></a>

e2e/reactive-directive-e2e.spec.ts

+1
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@ describe('dpDayPicker reactive directive', () => {
4646
});
4747

4848
it('should check if enable/disable required validation is working', () => {
49+
page.dayDirectiveReactiveInput.click();
4950
page.dayDirectiveReactiveInput.clear();
5051
expect(page.reactiveRequiredValidationMsg.isPresent()).toBe(false);
5152
page.enableRequiredValidationRadio.click();

e2e/timepicker-e2e.spec.ts

+1
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@ describe('dpDayPicker timePicker', () => {
2929
});
3030

3131
it('should check if max time validation is working', () => {
32+
page.maxTimeValidationPickerInput.click();
3233
page.maxTimeValidationPickerInput.clear();
3334
expect(page.timePickerMaxTimeValidationMsg.isPresent()).toBe(false);
3435
page.maxTimeValidationPickerInput.clear();

src/app/demo/demo/demo.component.html

-2
Original file line numberDiff line numberDiff line change
@@ -89,8 +89,6 @@ <h3 class="dp-menu-header">Components</h3>
8989

9090
<h2 class="dp-page-header">Angular Date/Time/Month Picker</h2>
9191

92-
<!--<div>{{form.form.dirty}}</div>-->
93-
9492
<form #form="ngForm">
9593
<div class="dp-picker-time-picker" *ngIf="pickerMode === 'daytimePicker'">
9694
<dp-date-picker id="daytimePicker"

0 commit comments

Comments
 (0)