feedback.txt

# Feedback

Group jackal: ikerumd, ruby7188

Commit hash: 88160a9b0b3c9876c9956450f2d48020227f797a

Final score: 66 / 71

## Style check

Score: 5 / 5

## ArrayDictionary

Score: 15 / 15

-   PASS: testPutAndGetSameKeyRepeatedMany
-   PASS: testRemoveDuplicate
-   PASS: testContainsKeyBasic
-   PASS: testRemoveBasic
-   PASS: testIteratorEndsCorrectly
-   PASS: testIteratorRunsMultipleTimes
-   PASS: testGetMany
-   PASS: testPutAndGetSameKeyRepeated
-   PASS: testNonNullKeys
-   PASS: testIteratorsAreIndependent
-   PASS: testIteratorOverEmptyDictionary
-   PASS: testPutAndGetBasic
-   PASS: testIteratorUnusualKeys
-   PASS: testAddGetMany
-   PASS: testIteratorOverDictionaryWithOneElement
-   PASS: testGetErrorHandling
-   PASS: testLargeKeys
-   PASS: testRemoveErrorHandling
-   PASS: testIterator
-   PASS: testNullKey
-   PASS: testPutAndGetMany

## ChainedHashDictionary

Score: 25 / 30

-   PASS: handlesNegativeHashCodesEfficiently
-   PASS: testInsertingDuplicatesDoesNotResize
-   PASS: testInternalStructure
-   PASS: testResizing
-   FAIL: testMethodIntegration

        java.util.NoSuchElementException
        	at datastructures.concrete.dictionaries.ArrayDictionary$ArrayDictionaryIterator.next(ArrayDictionary.java:172)
        	at datastructures.concrete.dictionaries.ArrayDictionary$ArrayDictionaryIterator.next(ArrayDictionary.java:155)
        	at datastructures.concrete.dictionaries.ChainedHashDictionary$ChainedIterator.next(ChainedHashDictionary.java:212)
        	at datastructures.concrete.dictionaries.ChainedHashDictionary$ChainedIterator.next(ChainedHashDictionary.java:167)
        	at privatetests.TestPrivateChainedHashDictionary.testMethodIntegration(TestPrivateChainedHashDictionary.java:128)
        	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
        	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        	at java.lang.reflect.Method.invoke(Method.java:498)
        	at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
        	at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
        	at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
        	at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
        	at org.junit.internal.runners.statements.FailOnTimeout$CallableStatement.call(FailOnTimeout.java:298)
        	at org.junit.internal.runners.statements.FailOnTimeout$CallableStatement.call(FailOnTimeout.java:292)
        	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
        	at java.lang.Thread.run(Thread.java:745)

-   PASS: testBigHashCodesAndNull
-   PASS: stressTest
-   PASS: testNegativeHashCode
-   PASS: testContainsMaintainsStateForIterator
-   PASS: testManyObjectsWithSameHashCode
-   PASS: testPutAndGetSameKeyRepeatedMany
-   PASS: testRemoveDuplicate
-   PASS: testContainsKeyBasic
-   PASS: testRemoveBasic
-   PASS: testIteratorEndsCorrectly
-   PASS: testIteratorRunsMultipleTimes
-   PASS: testGetMany
-   PASS: testPutAndGetSameKeyRepeated
-   PASS: testNonNullKeys
-   PASS: testIteratorsAreIndependent
-   PASS: testIteratorOverEmptyDictionary
-   PASS: testPutAndGetBasic
-   PASS: testIteratorUnusualKeys
-   PASS: testAddGetMany
-   PASS: testIteratorOverDictionaryWithOneElement
-   PASS: testGetErrorHandling
-   PASS: testLargeKeys
-   PASS: testRemoveErrorHandling
-   PASS: testIterator
-   PASS: testNullKey
-   PASS: testPutAndGetMany

## ChainedHashSet

Score: 10 / 10

-   PASS: testPutAndGetSameKeyRepeatedMany
-   PASS: testContainsWithRemovesBasic
-   PASS: testRemoveDuplicate
-   PASS: testContainsMany
-   PASS: testRemoveBasic
-   PASS: testIteratorEndsCorrectly
-   PASS: stressTest
-   PASS: testPutAndGetSameKeyRepeated
-   PASS: testContains
-   PASS: testIteratorsAreIndependent
-   PASS: testNegativeHashCode
-   PASS: testIteratorUnusualKeys
-   PASS: testAddAndCheckMany
-   PASS: testManyObjectsWithSameHashCode
-   PASS: testLargeKeys
-   PASS: testRemoveErrorHandling
-   PASS: testAddAndRemoveBasic
-   PASS: testIterator
-   PASS: testNullKey

## TestAvlTree

Score: 11 / 11