Remove name funciton from EffectSuite.

[zainab-ali]

All weaver suites have a publicly accessible name field corresponding to their class name.

This is confusing for users, who may reference it in their tests by mistake:

import weaver.*

object FooSuite extends SimpleIOSuite {
  pureTest("name should be foo") {
    val nam = "foo" // a typo
    expect("foo", name)
  }
}

This field should either be removed, or renamed to the more descriptive suiteName.