Create Check_Result class

[jjgrainger]

Adds the Check_Result class

• Addresses acceptance criteria in Create Check_Result class #11

Closes #11

[felixarntz]

@jjgrainger Left a few comments, mostly LGTM.

[felixarntz]

@jjgrainger Production code LGTM, left some feedback on the tests.

[felixarntz]

@jjgrainger Thanks, LGTM!

[felixarntz]

FYI no need to do any formal doc blocks in tests. It's good to add comments when a test method does things that may not be immediately clear on why it does so, but generally tests don't need docs.

Obviously not a blocker, it's okay to have this. But not needed :)

[mukeshpanchal27]

Thanks @jjgrainger The PR looks solid to me. I have only one suggestion regarding the count function.

Can we use the count function for errors and warnings? If yes, then it removes two variables from the class and the increment logic for those variables.

Ie.

public function get_error_count() {
    return count( $this->errors );
}

cc. @felixarntz

[jjgrainger]

Thanks @mukeshpanchal27

The structure of the errors and warnings arrays mean we cannot use the count() function and why we track the warning/errors count manually.