remove save from ui

Author: ShMcK

src/actions/runTest.ts

@@ -27,6 +27,7 @@ interface Props {
 }
 
 async function runTest({onSuccess, onFail, onRun, onError}: Props): Promise<void> {
+	console.log('------------------- run test ------------------')
 	// increment process id
 	const processId = ++currentId
 
@@ -44,6 +45,7 @@ async function runTest({onSuccess, onFail, onRun, onError}: Props): Promise<void
 	// jest CLI docs https://jestjs.io/docs/en/cli
 	const testArgs = [
 		'--json',
+		'--onlyChanged',
 		'--env=node',
 		'--maxConcurrency=4',
 		'--maxWorkers=4'

src/actions/tutorialConfig.ts

@@ -9,6 +9,7 @@ interface TutorialConfigParams {
 }
 
 const tutorialConfig = async ({tutorial, alreadyConfigured, onComplete}: TutorialConfigParams) => {
+	console.log('---------- tutorialConfig -----------')
 	if (!alreadyConfigured) {
 		// setup git, add remote
 		await git.initIfNotExists()

src/channel/index.ts

@@ -94,10 +94,6 @@ class Channel implements Channel {
 				this.context.position.set(action.payload.position)
 				this.context.progress.set(action.payload.progress)
 				return
-			// run unit tests on step
-			case 'TEST_RUN':
-				vscode.commands.executeCommand('coderoad.run_test', action.payload)
-				return
 			// load step actions (git commits, commands, open files)
 			case 'SETUP_ACTIONS':
 				vscode.commands.executeCommand('coderoad.set_current_step', action.payload)

src/editor/commands.ts

@@ -64,6 +64,7 @@ export const createCommands = ({extensionPath, workspaceState, workspaceRoot}: C
 			currentStepId = stepId
 		},
 		[COMMANDS.RUN_TEST]: (current: {stepId: string} | undefined) => {
+			console.log('-------- command.run_test ------ ')
 			// use stepId from client, or last set stepId
 			const payload = {stepId: current ? current.stepId : currentStepId}
 			runTest({
@@ -85,9 +86,9 @@ export const createCommands = ({extensionPath, workspaceState, workspaceRoot}: C
 					webview.send({type: 'TEST_ERROR', payload})
 				},
 				onRun: () => {
-					console.log('COMMAND TEST_RUN')
+					console.log('COMMAND TEST_RUNNING')
 					// send test run message back to client
-					webview.send({type: 'TEST_RUN', payload})
+					webview.send({type: 'TEST_RUNNING', payload})
 				}
 			})
 		},

web-app/src/containers/Tutorial/StagePage/Stage/index.tsx

@@ -25,11 +25,10 @@ const styles = {
 interface Props {
   stage: T.Stage
   onContinue(): void
-  onSave(): void
   onLoadSolution(): void
 }
 
-const Stage = ({ stage, onContinue, onSave, onLoadSolution }: Props) => {
+const Stage = ({ stage, onContinue, onLoadSolution }: Props) => {
   if (!stage.steps) {
     throw new Error('No Stage steps found')
   }
@@ -62,14 +61,10 @@ const Stage = ({ stage, onContinue, onSave, onLoadSolution }: Props) => {
         </Step>
       </div>
 
-      {stage.status === 'COMPLETE' ? (
+      {stage.status === 'COMPLETE' && (
         <div style={styles.options}>
           <Button onClick={onContinue}>Continue</Button>
         </div>
-      ) : (
-        <div style={styles.options}>
-          <Button onClick={onSave}>Save</Button>
-        </div>
       )}
     </div>
   )

web-app/src/containers/Tutorial/StagePage/index.tsx

@@ -24,15 +24,6 @@ const StageSummaryPageContainer = (props: PageProps) => {
     })
   }
 
-  const onSave = (): void => {
-    props.send({
-      type: 'TEST_RUN',
-      payload: {
-        stepId: position.stepId,
-      },
-    })
-  }
-
   const onLoadSolution = (): void => {
     props.send({ type: 'STEP_SOLUTION_LOAD' })
   }
@@ -48,7 +39,7 @@ const StageSummaryPageContainer = (props: PageProps) => {
   })
   stage.status = progress.stages[position.stageId] ? 'COMPLETE' : 'ACTIVE'
 
-  return <Stage stage={stage} onContinue={onContinue} onSave={onSave} onLoadSolution={onLoadSolution} />
+  return <Stage stage={stage} onContinue={onContinue} onLoadSolution={onLoadSolution} />
 }
 
 export default StageSummaryPageContainer

web-app/src/services/channel/index.ts

@@ -53,8 +53,7 @@ class Channel {
 			case 'TEST_FAIL':
 				this.machineSend(action)
 				return
-			case 'TEST_RUN':
-				console.log('TEST_RUN')
+			case 'TEST_RUNNING':
 				this.machineSend(action)
 				return
 			case 'TEST_ERROR':

web-app/src/services/state/actions/editor.ts

@@ -54,16 +54,16 @@ export default {
 			type: 'EDITOR_TUTORIAL_CONTINUE_CONFIG',
 		})
 	},
-	testStart(context: CR.MachineContext, event: CR.MachineEvent) {
-		console.log('EDITOR: TEST_RUN')
-		const {stepId} = event.payload
-		channel.editorSend({
-			type: 'TEST_RUN',
-			payload: {
-				stepId,
-			}
-		})
-	},
+	// testStart(context: CR.MachineContext, event: CR.MachineEvent) {
+	// 	console.log('EDITOR: TEST_RUN')
+	// 	const {stepId} = event.payload
+	// 	// channel.editorSend({
+	// 	// 	type: 'TEST_RUN',
+	// 	// 	payload: {
+	// 	// 		stepId,
+	// 	// 	}
+	// 	// })
+	// },
 	loadLevel(context: CR.MachineContext): void {
 		const level: G.Level = selectors.currentLevel(context)
 		if (level.setup) {

web-app/src/services/state/machine.ts

@@ -111,7 +111,7 @@ export const machine = Machine<CR.MachineContext, CR.MachineStateSchema, CR.Mach
 						states: {
 							Normal: {
 								on: {
-									TEST_RUN: 'TestRunning',
+									TEST_RUNNING: 'TestRunning',
 									STEP_SOLUTION_LOAD: {
 										actions: ['editorLoadSolution'],
 									},