Added some tests
authorKurt Pattyn <pattyn.kurt@gmail.com>
Sat, 5 Oct 2013 19:39:51 +0000 (21:39 +0200)
committerKurt Pattyn <pattyn.kurt@gmail.com>
Sat, 5 Oct 2013 20:09:39 +0000 (22:09 +0200)
Change-Id: I598de5c955eb8c5224035fd9c2469b132c39d06c
Reviewed-by: Kurt Pattyn <pattyn.kurt@gmail.com>
tests/auto/dataprocessor/tst_dataprocessor.cpp

index 1ab0ece..7fe384d 100644 (file)
@@ -14,9 +14,9 @@ Q_DECLARE_METATYPE(QWebSocketProtocol::OpCode)
 //TODO: test valid UTF8 sequences (see UC 6.2)
 //TODO: test on masking correctness
 //TODO: test for valid fields
-//TODO: test for valid opcodes
+//TODO: test for valid control frames
 //DONE: test for valid close codes
-//TODO: test close frame with no close code and reason
+//DONE: test close frame with no close code and reason
 //TODO: test if opcode is correct after processing of a continuation frame (text and binary frames)
 
 //TODO: test valid frame sequences
@@ -241,7 +241,6 @@ void tst_DataProcessor::goodBinaryFrame_data()
 
 void tst_DataProcessor::goodCloseFrame_data()
 {
-    //TODO: still to test with no close code and no close reason (is valid)
     QTest::addColumn<QString>("payload");
     QTest::addColumn<QWebSocketProtocol::CloseCode>("closeCode");
     //control frame data cannot exceed 125 bytes; smaller than 124, because we also need a 2 byte close code