Thanks to visit codestin.com
Credit goes to github.com

Skip to content

xUnit report test/suite name of description is null #370

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

Merged
jgebal merged 1 commit into from
Jun 27, 2017
Merged

xUnit report test/suite name of description is null #370

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
fixed issue #359
  • Loading branch information
@Pazus
Pazus committed Jun 27, 2017
commit 7ecfd21eec0d0716e11289c435fbc8f2f88e6ab8
2 changes: 1 addition & 1 deletion source/reporters/ut_xunit_reporter.tpb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -122,7 +122,7 @@ create or replace type body ut_xunit_reporter is

member function get_common_item_attributes(a_item ut_suite_item) return varchar2 is
begin
return ' skipped="' || a_item.results_count.disabled_count || '" error="' || a_item.results_count.errored_count || '"' || ' failure="' || a_item.results_count.failure_count || '" name="' || a_item.description || '"' || ' time="' || a_item.execution_time() || '" ';
return ' skipped="' || a_item.results_count.disabled_count || '" error="' || a_item.results_count.errored_count || '"' || ' failure="' || a_item.results_count.failure_count || '" name="' || nvl(a_item.description, a_item.name) || '"' || ' time="' || a_item.execution_time() || '" ';
end;

end;
Expand Down
2 changes: 2 additions & 0 deletions tests/RunAll.sql
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -134,6 +134,8 @@ exec ut_coverage.coverage_start_develop();
@@lib/RunTest.sql ut_reporters/ut_xunit_reporter.ProducesExpectedOutputs.sql
@@lib/RunTest.sql ut_reporters/ut_html_reporter.UserOverrideSchemaCoverage.sql
@@lib/RunTest.sql ut_reporters/ut_html_reporter.DefaultSchemaCoverage.sql
@@lib/RunTest.sql ut_reporters/ut_xunit_reporter.ReportOnSuiteWithoutDesc.sql
@@lib/RunTest.sql ut_reporters/ut_xunit_reporter.ReportOnTestWithoutDesc.sql

@@lib/RunTest.sql ut/ut.run.AcceptsCoverageFileList.sql
@@lib/RunTest.sql ut/ut.run.AcceptsCoverageFileListWithSutePaths.sql
Expand Down
47 changes: 47 additions & 0 deletions tests/ut_reporters/ut_xunit_reporter.ReportOnSuiteWithoutDesc.sql
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,47 @@
set termout off
create or replace package tst_package_xunit_nodesc as
--%suite

--%test(Test name)
procedure test1;
end;
/

create or replace package body tst_package_xunit_nodesc as
procedure test1 is begin ut.expect(1).to_equal(1); end;
procedure test2 is begin ut.expect(1).to_equal(1); end;
end;
/

set termout on

declare
l_test_report ut_varchar2_list;
l_output_data ut_varchar2_list;
l_output varchar2(32767);
l_expected varchar2(32767);
begin
l_expected := q'[<testsuites tests="1" skipped="0" error="0" failure="0" name="" time="%" >
<testsuite tests="1" id="1" package="tst_package_xunit_nodesc" skipped="0" error="0" failure="0" name="tst_package_xunit_nodesc" time="%" >
<testcase classname="tst_package_xunit_nodesc" assertions="1" skipped="0" error="0" failure="0" name="Test name" time="%" >
</testcase>
</testsuite>
</testsuites>]';

--act
select *
bulk collect into l_output_data
from table(ut.run('tst_package_xunit_nodesc',ut_xunit_reporter()));

l_output := ut_utils.table_to_clob(l_output_data);

--assert
if l_output like l_expected then
:test_result := ut_utils.tr_success;
else
dbms_output.put_line('Actual:"'||l_output||'"');
end if;
end;
/

drop package tst_package_xunit_nodesc;
52 changes: 52 additions & 0 deletions tests/ut_reporters/ut_xunit_reporter.ReportOnTestWithoutDesc.sql
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,52 @@
set termout off
create or replace package tst_package_xunit_nodesc as
--%suite(Suite name)

--%test
procedure test1;

--%test(Test name)
procedure test2;
end;
/

create or replace package body tst_package_xunit_nodesc as
procedure test1 is begin ut.expect(1).to_equal(1); end;
procedure test2 is begin ut.expect(1).to_equal(1); end;
end;
/

set termout on

declare
l_test_report ut_varchar2_list;
l_output_data ut_varchar2_list;
l_output varchar2(32767);
l_expected varchar2(32767);
begin
l_expected := q'[<testsuites tests="2" skipped="0" error="0" failure="0" name="" time="%" >
<testsuite tests="2" id="1" package="tst_package_xunit_nodesc" skipped="0" error="0" failure="0" name="Suite name" time="%" >
<testcase classname="tst_package_xunit_nodesc" assertions="1" skipped="0" error="0" failure="0" name="test1" time="%" >
</testcase>
<testcase classname="tst_package_xunit_nodesc" assertions="1" skipped="0" error="0" failure="0" name="Test name" time="%" >
</testcase>
</testsuite>
</testsuites>]';

--act
select *
bulk collect into l_output_data
from table(ut.run('tst_package_xunit_nodesc',ut_xunit_reporter()));

l_output := ut_utils.table_to_clob(l_output_data);

--assert
if l_output like l_expected then
:test_result := ut_utils.tr_success;
else
dbms_output.put_line('Actual:"'||l_output||'"');
end if;
end;
/

drop package tst_package_xunit_nodesc;