Looks good! Associated values are great if we want to save some specific information along with an enumeration.
Again, does that look like the string value we require the associated value to have?