Limit chat message size to 8192 bytes

[rbdr/mobius] / hotline / flattened_file_object_test.go

diff --git a/hotline/flattened_file_object_test.go b/hotline/flattened_file_object_test.go
index 300eb343c3f0d7ee31a0b3206d999961458a9a64..274ca2b1831ef5552f704d56739cc945d077c6f9 100644 (file)
--- a/hotline/flattened_file_object_test.go
+++ b/hotline/flattened_file_object_test.go
@@ -3,65 +3,41 @@ package hotline
 import (
        "fmt"
        "github.com/stretchr/testify/assert"
 import (
        "fmt"
        "github.com/stretchr/testify/assert"

-       "os"

        "testing"
 )
 
        "testing"
 )
 

-func TestNewFlattenedFileObject(t *testing.T) {
+func TestFlatFileInformationFork_UnmarshalBinary(t *testing.T) {

        type args struct {
        type args struct {

-               fileRoot string
-               filePath []byte
-               fileName []byte
+               b []byte

        }
        tests := []struct {
                name    string
                args    args
        }
        tests := []struct {
                name    string
                args    args

-               want    *flattenedFileObject

                wantErr assert.ErrorAssertionFunc
        }{
                {
                wantErr assert.ErrorAssertionFunc
        }{
                {

-                       name: "with valid file",
+                       name: "when zero length comment size is omitted (Nostalgia client behavior)",

                        args: args{
                        args: args{

-                               fileRoot: func() string { path, _ := os.Getwd(); return path + "/test/config/Files" }(),
-                               fileName: []byte("testfile.txt"),
-                               filePath: []byte{0, 0},
-                       },
-                       want: &flattenedFileObject{
-                               FlatFileHeader:                NewFlatFileHeader(),
-                               FlatFileInformationForkHeader: FlatFileInformationForkHeader{},
-                               FlatFileInformationFork:       NewFlatFileInformationFork("testfile.txt", make([]byte, 8)),
-                               FlatFileDataForkHeader: FlatFileDataForkHeader{
-                                       ForkType:        [4]byte{0x4d, 0x41, 0x43, 0x52}, // DATA
-                                       CompressionType: [4]byte{0, 0, 0, 0},
-                                       RSVD:            [4]byte{0, 0, 0, 0},
-                                       DataSize:        [4]byte{0x00, 0x00, 0x00, 0x17},
+                               b: []byte{
+                                       0x41, 0x4d, 0x41, 0x43, 0x3f, 0x3f, 0x3f, 0x3f, 0x3f, 0x3f, 0x3f, 0x3f, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x09, 0x62, 0x65, 0x61, 0x72, 0x2e, 0x74, 0x69, 0x66, 0x66,

                                },
                                },

-                               FileData: nil,

                        },
                        wantErr: assert.NoError,
                },
                {
                        },
                        wantErr: assert.NoError,
                },
                {

-                       name: "when file path is invalid",
+                       name: "when zero length comment size is included",

                        args: args{
                        args: args{

-                               fileRoot: func() string { path, _ := os.Getwd(); return path + "/test/config/Files" }(),
-                               fileName: []byte("nope.txt"),
+                               b: []byte{
+                                       0x41, 0x4d, 0x41, 0x43, 0x3f, 0x3f, 0x3f, 0x3f, 0x3f, 0x3f, 0x3f, 0x3f, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x09, 0x62, 0x65, 0x61, 0x72, 0x2e, 0x74, 0x69, 0x66, 0x66, 0x00, 0x00,
+                               },

                        },
                        },

-                       want:    nil,
-                       wantErr: assert.Error,
+                       wantErr: assert.NoError,

                },
        }
        for _, tt := range tests {
                t.Run(tt.name, func(t *testing.T) {
                },
        }
        for _, tt := range tests {
                t.Run(tt.name, func(t *testing.T) {

-                       got, err := NewFlattenedFileObject(tt.args.fileRoot, tt.args.filePath, tt.args.fileName)
-                       if tt.wantErr(t, err, fmt.Sprintf("NewFlattenedFileObject(%v, %v, %v)", tt.args.fileRoot, tt.args.filePath, tt.args.fileName)) {
-                               return
-                       }
-
-                       // Clear the file timestamp fields to work around problems running the tests in multiple timezones
-                       // TODO: revisit how to test this by mocking the stat calls
-                       got.FlatFileInformationFork.CreateDate = make([]byte, 8)
-                       got.FlatFileInformationFork.ModifyDate = make([]byte, 8)
-                       assert.Equalf(t, tt.want, got, "NewFlattenedFileObject(%v, %v, %v)", tt.args.fileRoot, tt.args.filePath, tt.args.fileName)
+                       ffif := &FlatFileInformationFork{}
+                       tt.wantErr(t, ffif.UnmarshalBinary(tt.args.b), fmt.Sprintf("Write(%v)", tt.args.b))

                })
        }
 }
                })
        }
 }