Skip to content

Commit f9c84f1

Browse files
committed
Merge remote-tracking branch 'origin/main' into develop
2 parents 955f8bf + a191427 commit f9c84f1

File tree

4 files changed

+38
-72
lines changed

4 files changed

+38
-72
lines changed

.github/workflows/build.yml

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,9 @@ on:
77
branches: [ develop ]
88

99
workflow_dispatch:
10+
11+
repository_dispatch:
12+
type: [utPLSQL-build]
1013

1114
jobs:
1215
build:
@@ -74,3 +77,19 @@ jobs:
7477
if: always()
7578
with:
7679
files: target/**/TEST**.xml
80+
81+
slack-workflow-status:
82+
if: always()
83+
name: Post Workflow Status To Slack
84+
needs: [ build ]
85+
runs-on: ubuntu-latest
86+
steps:
87+
- name: Slack Workflow Notification
88+
uses: Gamesight/slack-workflow-status@master
89+
with:
90+
# Required Input
91+
repo_token: ${{secrets.GITHUB_TOKEN}}
92+
slack_webhook_url: ${{secrets.SLACK_WEBHOOK_URL}}
93+
# Optional Input
94+
name: 'Github Actions[bot]'
95+
icon_url: 'https://octodex.github.com/images/mona-the-rivetertocat.png'

.github/workflows/release.yml

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,3 +56,19 @@ jobs:
5656
MAVEN_USERNAME: ${{ secrets.OSSRH_USERNAME }}
5757
MAVEN_PASSWORD: ${{ secrets.OSSRH_TOKEN }}
5858
MAVEN_GPG_PASSPHRASE: ${{ secrets.MAVEN_GPG_PASSPHRASE }}
59+
60+
slack-workflow-status:
61+
if: always()
62+
name: Post Workflow Status To Slack
63+
needs: [ build ]
64+
runs-on: ubuntu-latest
65+
steps:
66+
- name: Slack Workflow Notification
67+
uses: Gamesight/slack-workflow-status@master
68+
with:
69+
# Required Input
70+
repo_token: ${{secrets.GITHUB_TOKEN}}
71+
slack_webhook_url: ${{secrets.SLACK_WEBHOOK_URL}}
72+
# Optional Input
73+
name: 'Github Actions[bot]'
74+
icon_url: 'https://octodex.github.com/images/mona-the-rivetertocat.png'

src/test/java/org/utplsql/maven/plugin/UtPlsqlMojoTest.java

Lines changed: 3 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -306,11 +306,11 @@ public void db_config_using_system_properties() throws Exception {
306306
}
307307

308308
/**
309-
* Exclude object from coverage report
309+
* DB configuration from System Properties
310310
* <p>
311-
* Given : a pom.xml with excludeObject set
311+
* Given : a pom.xml without dbUrl, dbUser and dbPass configured
312312
* When : pom is read
313-
* Then : excludeObject is present
313+
* Then : System Properties must be used to configure database
314314
*/
315315
@Test
316316
public void exclude_object() throws Exception {
@@ -322,23 +322,6 @@ public void exclude_object() throws Exception {
322322
assertEquals("app.pkg_test_me,app.test_pkg_test_me", utPlsqlMojo.excludeObject);
323323
}
324324

325-
/**
326-
* Include object from coverage report
327-
* <p>
328-
* Given : a pom.xml with includeObject set
329-
* When : pom is read
330-
* Then : includeObject is present
331-
*/
332-
@Test
333-
public void include_object() throws Exception {
334-
UtPlsqlMojo utPlsqlMojo = createUtPlsqlMojo("include_object");
335-
assertNotNull(utPlsqlMojo);
336-
337-
utPlsqlMojo.execute();
338-
339-
assertEquals("app.pkg_test_me,app.test_pkg_test_me", utPlsqlMojo.includeObject);
340-
}
341-
342325
private UtPlsqlMojo createUtPlsqlMojo(String directory) throws Exception {
343326
return (UtPlsqlMojo) rule.lookupConfiguredMojo(new File("src/test/resources/unit-tests/" + directory), "test");
344327
}

src/test/resources/unit-tests/include_object/pom.xml

Lines changed: 0 additions & 52 deletions
This file was deleted.

0 commit comments

Comments
 (0)