Woah! This is a lot of class definitions! That is a bit longwinded, and I feel that we should be testing the actual classes body that we care about: classes_generic.

Also, are you sure it is legal in CFEngine to define the classes attribute multiple times on one promise? I'm not sure...

I think you should instead be using ncf as it is intended, and testing with other generic methods. Just some that necessarily fail, or repair something pointless (editing a file) or are necessarly in success.

For example:

```
methods:
  "ph" usemodule => file_ensure_lines_absent("${init.file}", "dsdsds"); # should get success
  "ph" usemodule => file_ensure_lines_present("${init.file}", "test content"); # should get repaired
  "ph" usemodule => file_ensure_lines_absent("/dev/zero", "blablabla"); # should get failed
```