Opened 7 years ago

Closed 7 years ago

#5573 closed Enhancement (fixed)

Crypto unit test

Reported by: mike.dld Owned by: jordan
Priority: Normal Milestone: 2.83
Component: libtransmission Version: 2.82
Severity: Normal Keywords:
Cc:

Description

In the light of #4400 and #5462, it would be wise to first add unit test for crypto functions before making any modifications. Attached is my first take on it, let me know if some other cases are needed.

Attachments (1)

5573-crypto-test.patch (5.1 KB) - added by mike.dld 7 years ago.

Download all attachments as: .zip

Change History (3)

Changed 7 years ago by mike.dld

comment:1 Changed 7 years ago by jordan

  • Milestone changed from None Set to 2.83
  • Status changed from new to assigned

Looks like a good attempt. It's not like we can do meaningful testing of tr_cryptoRand*()... :)

comment:2 Changed 7 years ago by jordan

  • Resolution set to fixed
  • Status changed from assigned to closed

Added to trunk in r14233.

Note: See TracTickets for help on using tickets.