Project

General

Profile

Actions

Task #3514

closed

Merge Channel test cases as template

Added by Junxiao Shi over 8 years ago. Updated about 8 years ago.

Status:
Closed
Priority:
Normal
Category:
Faces
Target version:
Start date:
Due date:
% Done:

100%

Estimated time:
3.00 h

Description

Merge similar test cases from UnixStreamChannel TcpChannel UdpChannel WebSocketChannel test suites into test case templates.


Related issues 4 (0 open4 closed)

Blocked by NFD - Task #3372: UnixStreamChannel: test suite improvementClosedSpencer Lee

Actions
Blocked by NFD - Task #3370: TcpChannel: test suite improvementClosedSpencer Lee

Actions
Blocked by NFD - Task #3371: UdpChannel: test suite improvementClosedSpencer Lee

Actions
Blocked by NFD - Task #3373: WebSocketChannel: test suite improvementClosedWeiwei Liu

Actions
Actions #1

Updated by Junxiao Shi over 8 years ago

  • Blocked by Task #3372: UnixStreamChannel: test suite improvement added
Actions #2

Updated by Junxiao Shi over 8 years ago

  • Blocked by Task #3370: TcpChannel: test suite improvement added
Actions #3

Updated by Junxiao Shi over 8 years ago

  • Blocked by Task #3371: UdpChannel: test suite improvement added
Actions #4

Updated by Junxiao Shi over 8 years ago

  • Blocked by Task #3373: WebSocketChannel: test suite improvement added
Actions #5

Updated by Junxiao Shi over 8 years ago

  • Assignee set to Davide Pesavento

@Davide agrees to work on this in #3371 note-9.
This Task does not block 0.4.1 release.

Actions #6

Updated by Davide Pesavento over 8 years ago

Where should the test case templates reside? channel.t.cpp?

Actions #7

Updated by Alex Afanasyev over 8 years ago

How much duplication we have (i.e., is it worth effort)?

I would call it "channel-common.t.cpp", to highlight that it is not generic "channel" tests, but common tests of different channels. Though, name doesn't matter much.

Actions #8

Updated by Davide Pesavento over 8 years ago

Alex Afanasyev wrote:

How much duplication we have (i.e., is it worth effort)?

TcpChannel and UdpChannel are practically identical. WebSocketChannel uses a substantially different model, so I probably won't be able to merge it. I haven't looked at UnixStreamChannel yet.

I would call it "channel-common.t.cpp", to highlight that it is not generic "channel" tests, but common tests of different channels. Though, name doesn't matter much.

Sounds good.

Actions #9

Updated by Davide Pesavento about 8 years ago

  • Status changed from New to In Progress
Actions #10

Updated by Davide Pesavento about 8 years ago

  • Status changed from In Progress to Code review
  • % Done changed from 0 to 100

I was able to merge TcpChannel and UdpChannel suites relatively easily. UnixStreamChannel and WebSocketChannel have some deep differences that are not easy to overcome without making the code much more complex (and probably unreadable), albeit they do have a couple of test cases that look similar. I don't think trying to merge them is worth the effort though, and even then we would still have some special cases for each channel type.

Actions #11

Updated by Davide Pesavento about 8 years ago

  • Status changed from Code review to Closed
Actions

Also available in: Atom PDF