Changes between Version 2 and Version 3 of Ticket #41722, comment 17
- Timestamp:
- 10/10/2017 07:08:38 PM (6 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #41722, comment 17
v2 v3 4 4 5 5 - Replace {{{assertEquals()}}} with {{{assertSame()}}} 6 - Added the same empty check for {{{$ socket}}}, as done in [attachment:41722-sockets-fix.diff] for the {{{$port}}}.6 - Added the same empty check for {{{$host}}} and {{{$socket}}}, as done in [attachment:41722-sockets-fix.diff] for the {{{$port}}}. 7 7 8 8 In the first test, within {{{parse_db_host_data_provider()}}}, the empty string {{{DB_HOST}}} is expected to be parsed into a {{{null}}} {{{$host}}}. So this patch assumes that output for empty {{{$host}}}. It would be good to have others verifying that this is valid.