lotteryStatus

[aklein-dex]

Hey guys,
I'm new to solidity and I was looking at your code to try to understand how things work.
Anyway, isn't this test incorrect:

lottery-contract/contracts/Lottery.sol

Line 430 in d2ebe90

if(_startingTimestamp >= getCurrentTime()) {

if(_startingTimestamp >= getCurrentTime()) {
     lotteryStatus = Status.Open;
} else {
      lotteryStatus = Status.NotStarted;
}

If the starting timestamp is bigger than the current time, then the status should be NotStarted. It should be like this:

if(_startingTimestamp >= getCurrentTime()) {
     lotteryStatus = Status.NotStarted;
} else {
      lotteryStatus = Status.Open;
}

And maybe I would add a "require" to make sure that _closingTimestamp is bigger than getCurrentTime(). Because there is no point to create a lottery that is already closed.