Updated descriptor names-review comment.

Author: nbhoski

src/main/java/com/mathworks/ci/AddMatlabToPathBuildWrapper.java

@@ -56,7 +56,7 @@ private void setEnv(EnvVars env) {
 
     @Symbol("Matlab")
     @Extension
-    public static final class MatlabBuildWrapperDescriptor extends BuildWrapperDescriptor {
+    public static final class AddMatlabToPathDescriptor extends BuildWrapperDescriptor {
 
         MatlabReleaseInfo rel;
         String matlabRootFolder;

src/main/java/com/mathworks/ci/RunMatlabCommandBuilder.java

@@ -64,7 +64,7 @@ private EnvVars getEnv() {
 
     @Symbol("RunMatlabCommand")
     @Extension
-    public static class MatlabScriptDescriptor extends BuildStepDescriptor<Builder> {
+    public static class RunMatlabCommandDescriptor extends BuildStepDescriptor<Builder> {
 
         // Overridden Method used to show the text under build dropdown
         @Override
@@ -114,8 +114,6 @@ private synchronized int execMatlabCommand(FilePath workspace, Launcher launcher
             TaskListener listener, EnvVars envVars) throws IOException, InterruptedException {
         ProcStarter matlabLauncher;
         try {
-            // Get mMatlabroot set in wrapper.
-            String matlabRoot = envVars.get("matlabroot");
             matlabLauncher = launcher.launch().pwd(workspace).envs(envVars);
             FilePath targetWorkspace = new FilePath(launcher.getChannel(), workspace.getRemote());
             if(launcher.isUnix()) {

src/main/java/com/mathworks/ci/RunMatlabTestsBuilder.java

@@ -21,7 +21,7 @@
 import org.kohsuke.stapler.DataBoundSetter;
 import org.kohsuke.stapler.QueryParameter;
 import org.kohsuke.stapler.StaplerRequest;
-import com.mathworks.ci.AddMatlabToPathBuildWrapper.MatlabBuildWrapperDescriptor;
+import com.mathworks.ci.AddMatlabToPathBuildWrapper.AddMatlabToPathDescriptor;
 import hudson.EnvVars;
 import hudson.Extension;
 import hudson.FilePath;
@@ -124,7 +124,7 @@ private EnvVars getEnv() {
 
     @Symbol("RunMatlabTests")
     @Extension
-    public static class MatlabTestDescriptor extends BuildStepDescriptor<Builder> {
+    public static class RunMatlabTestsDescriptor extends BuildStepDescriptor<Builder> {
 
         MatlabReleaseInfo rel;
 
@@ -247,7 +247,7 @@ public FormValidation getFirstErrorOrWarning(
                 return FormValidation.ok();
             try {
                 final String matlabRoot = Jenkins.getInstance()
-                        .getDescriptorByType(MatlabBuildWrapperDescriptor.class).getMatlabRootFolder();
+                        .getDescriptorByType(AddMatlabToPathDescriptor.class).getMatlabRootFolder();
                 for (Function<String, FormValidation> val : validations) {
                     FormValidation validationResult = val.apply(matlabRoot);
                     if (validationResult.kind.compareTo(Kind.ERROR) == 0
@@ -284,8 +284,6 @@ private synchronized int execMatlabCommand(FilePath workspace, Launcher launcher
             TaskListener listener, EnvVars envVars) throws IOException, InterruptedException {
         ProcStarter matlabLauncher;
         try {
-            // Get matlabroot set in wrapper class.
-            String matlabRoot = envVars.get("matlabroot");
             matlabLauncher = launcher.launch().pwd(workspace).envs(envVars);
             FilePath targetWorkspace = new FilePath(launcher.getChannel(), workspace.getRemote());
             if (launcher.isUnix()) {