Add LPUSH, RPUSH, LPOP, RPOP, LLEN, LMOVE, LTRIM and tests

[cchriss123]

Methods still need to be called correctly from main or somewhere else.
#101

[safstromo]

Great job, so much code :D

This test is failing, assertEquals("$3\r\n(nil)\r\n", nilTest); $3 should be $5 since (nil) is 5 letters.

I think you need to fix lmove, how im reading the documentation val1 should be push to the front of the list?

[cchriss123]

@safstromo
Great catch on the nil Test.
I forgot to update the test when i found the error in my code last night.
I will have to look into the LMOVE closer but in the picture you provided you are comparing a LEFT to RIGHT with a RIGHT to LEFT if im not mistaken.

[safstromo]

@safstromo Great catch on the nil Test. I forgot to update the test when i found the error in my code last night. I will have to look into the LMOVE closer but in the picture you provided you are comparing a LEFT to RIGHT with a RIGHT to LEFT if im not mistaken.

Your right my mistake! Move along nothing to see here xD

[safstromo]

👍

[RehasLiamsi]

Looks good to me

[Viktormes]

Should rename the methods. Should be using the Java Naming Conventions.

[safstromo]

You still have some code smells :)

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
9 Code Smells

93.2% Coverage
0.0% Duplication

[BobHogvall]

This is really coming along great! Just a small reminder to not forget to look at sonarCloud! there are still a few code smells from what i can see.
Good work Christofer!

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

94.8% Coverage
0.0% Duplication