-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
UI testing PoC with the Lombiq UI Testing Toolbox (Lombiq Technologies: OCORE-75) #11194
Changes from 63 commits
1c753df
e2f0900
224ad84
7e6353c
2be799d
22f2acb
4d12b4c
62663cb
6842a5b
6b4b044
79562ff
580ae14
8c04eab
c0c823f
b09fdb1
2035114
9da0d19
37f4a00
86f9444
f2bfaa6
b8203f0
2ac49a2
569a7f8
3b56899
47d0c01
48cda29
ce772df
5602051
73da5c6
5692272
181f30a
81e6587
2bca0d1
c62003e
0b31023
06e71c7
58b2f15
3e83441
bb81d83
ac5cf10
ff075c5
d6f4cef
4e0c768
99cb01e
23df70d
4ee6220
2872167
6799072
36d33b6
6880e6e
dd58831
6ebf938
c1300f8
8f562cb
0ae18bf
f847dca
0100bc9
91db2ab
b7e50fe
046d397
25b7372
8fcbe9a
f6b9429
a389874
0a79906
d56c708
f4ca49d
7aebfcb
4d8d564
50e9d4e
7fe7433
dcf9433
e80515f
13f4f90
015c2f5
9cf30c8
212949b
6ef7b4e
359a313
3255e4b
14335d6
e562ace
d15e1a0
c980623
826fc6d
282b166
2384df1
2bf138b
41a5b3e
e9c8a13
8359e9f
0886b05
ac80b83
5397c14
87698ff
6eb18cd
5dedbd0
3cdca82
f2ac898
3eaac46
7c55f46
e9c6b82
20a485c
acc1699
3f4b18b
ea2cc02
78c2539
4d75586
9e1d1a3
841c2fa
7fedb67
3b002a0
b0f0e9e
bbb710f
1f2cd9c
c3ada4b
1011d61
e2b81f1
80deb0c
e57f590
bf7064a
eb9b117
35d5422
19d8711
fd62ab5
3836bbf
d462e1f
10b84b5
b123bde
a151302
119d8d7
320d173
9399474
9515fa8
95179cc
8b9da4d
f3c0588
40b6bee
9c2b48a
5adc68d
00d0c04
b8663eb
21a6bd3
26e62e2
57b771b
55f4500
9aca1fb
291364f
9fb42a8
a0f6501
4cc769c
8eea409
3ff0887
0708104
61427f8
97b0613
8ae3845
ff29a39
166e906
dee49ba
c3050d0
9b55db4
0e9a358
0e4a3dd
ceab4be
398f203
a3233d0
fd33317
86dae36
47f1497
0eaf101
9d07d35
f29b0fa
6c2d033
6eb0e7f
1cea784
44e39f9
8d67700
345dea0
c04ecda
41e9057
a4bdfa1
756378a
b9f4bc1
e1b87e2
9ba9ab7
daa0fdc
9c56fb4
eaeb663
ce0dd0a
8125eb0
6b2df55
4c1f070
7087ac7
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,21 @@ | ||
name: UI Tests | ||
description: Runs UI tests and publishes the failure dumps as an artifact on test failure. | ||
|
||
inputs: | ||
os-name: | ||
description: The name of the OS used for the runner (e.g. "ubuntu-latest"). | ||
required: true | ||
|
||
runs: | ||
using: "composite" | ||
steps: | ||
- name: UI Tests | ||
shell: bash | ||
run: | | ||
dotnet test -c Release --no-restore --no-build ./test/OrchardCore.Tests.UI/OrchardCore.Tests.UI.csproj | ||
- uses: actions/upload-artifact@v2 | ||
if: failure() && hashFiles('test/OrchardCore.Tests.UI/bin/Release/*/FailureDumps') | ||
with: | ||
name: ui-test-failure-${{ inputs.os-name }} | ||
path: | | ||
test/OrchardCore.Tests.UI/bin/Release/*/FailureDumps |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -15,6 +15,8 @@ | |
<target xsi:type="File" name="file" | ||
fileName="${var:configDir}/App_Data/logs/orchard-log-${shortdate}.log" | ||
layout="${longdate}|${orchard-tenant-name}|${aspnet-traceidentifier}|${event-properties:item=EventId.Id}|${logger}|${uppercase:${level}}|${message} ${exception:format=ToString,StackTrace}" | ||
keepFileOpen="false" | ||
concurrentWrites="false" | ||
Comment on lines
+18
to
+19
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This should be reverted once we have a better approach, see Lombiq/Open-Source-Orchard-Core-Extensions#264 (comment). |
||
/> | ||
|
||
<!-- console target --> | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -21,7 +21,7 @@ | |
|
||
<!-- Watcher include and excludes --> | ||
<ItemGroup> | ||
<Watch Include="**\*.cs" Exclude="Recipes\**;Assets\**;node_modules\**\*;**\*.js.map;obj\**\*;bin\**\*" /> | ||
<Watch Include="**\*.cs" Exclude="Recipes\**;Assets\**;node_modules\**\*;**\*.js.map;obj\**\*;bin\**\*" /> | ||
</ItemGroup> | ||
|
||
<ItemGroup> | ||
|
@@ -43,5 +43,9 @@ | |
<Import Project="..\OrchardCore\OrchardCore.Application.Targets\OrchardCore.Application.Targets.targets" /> | ||
|
||
<Import Project="..\OrchardCore.Build\OrchardCore.Commons.targets" /> | ||
|
||
<ItemGroup> | ||
<PackageReference Include="Lombiq.Tests.UI.Shortcuts" /> | ||
</ItemGroup> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Could we not build a separate csproj for tests, i.e. OracheCore.Cms.Web.UITests.csproj or something.... rather than adding in stuff to the main csproj for testing? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We could, but it would be a complete copy of this one, and then we'd have to maintain two. |
||
|
||
</Project> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you want to remove the Cypress functional tests?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Eventually, yes, this is a direct substitute.