-
Notifications
You must be signed in to change notification settings - Fork 36
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Tests implementation liberty-ls support in server.xml for diagnostic and quickfix #377
Open
SuparnaSuresh
wants to merge
35
commits into
OpenLiberty:main
Choose a base branch
from
SuparnaSuresh:issue124-AutomatedTestcaseLCLS
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
35 commits
Select commit
Hold shift + click to select a range
08aefcd
code to open server.xml and entry for logging
SuparnaSuresh c1a82a4
code added for hover and quicfix
SuparnaSuresh 121f17b
Code added to open server.xml
SuparnaSuresh 2771093
Code Added for write text in server.xml
SuparnaSuresh 80d8e87
Code for hover data
SuparnaSuresh 01d668f
Code for quick fix
SuparnaSuresh 3f5e62f
Code formatted
SuparnaSuresh dbba2a0
removed unwanted code
SuparnaSuresh 3e4affe
Copyright and docuementation
SuparnaSuresh 883f757
Merge branch 'OpenLiberty:main' into issue124-AutomatedTestcaseLCLS
SuparnaSuresh 150019c
Renamed file
SuparnaSuresh 7fdc278
Renamed file to GradleSingleModLCLSTest
SuparnaSuresh 108bfd5
added code to update the old content
SuparnaSuresh f238c52
updated quick fix element name
SuparnaSuresh 41114d8
Timeout value updated
SuparnaSuresh fab341b
Timeout value updated
SuparnaSuresh 26b5cdb
GHA build fix - timeout value updated
SuparnaSuresh 8986c67
Merge branch 'OpenLiberty:main' into issue124-AutomatedTestcaseLCLS
SuparnaSuresh 7c60cd8
Added mpHealth
SuparnaSuresh e7490fa
Merge branch 'OpenLiberty:main' into issue124-AutomatedTestcaseLCLS
SuparnaSuresh 13498f5
await added for find element
SuparnaSuresh 7ee5da3
Merge branch 'OpenLiberty:main' into issue124-AutomatedTestcaseLCLS
SuparnaSuresh 499247d
updated server.xml file
SuparnaSuresh 9c3b83a
updated - added code for new config folder
SuparnaSuresh 4d2fa91
updated the function removeConfigDir
SuparnaSuresh 3515049
server.env file added
SuparnaSuresh e320f4d
Merge branch 'OpenLiberty:main' into issue124-AutomatedTestcaseLCLS
SuparnaSuresh a582ac6
GHA build failure - Increased timeout value
SuparnaSuresh b193cc5
Added test case to coppy server.xml content
SuparnaSuresh 288d1e0
Diagnostic server.xml test case added
SuparnaSuresh 4492afc
updated test case
SuparnaSuresh 45f2c84
changed server.xml path
SuparnaSuresh 2cdd24d
Merge branch 'OpenLiberty:main' into issue124-AutomatedTestcaseLCLS
SuparnaSuresh 80d2951
addressed review comment
SuparnaSuresh a3ffad9
addressed review comments
SuparnaSuresh File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,122 @@ | ||
/** | ||
* Copyright (c) 2024 IBM Corporation. | ||
* | ||
* This program and the accompanying materials are made available under the | ||
* terms of the Eclipse Public License v. 2.0 which is available at | ||
* http://www.eclipse.org/legal/epl-2.0. | ||
* | ||
* SPDX-License-Identifier: EPL-2.0 | ||
*/ | ||
import { By, EditorView, SideBarView, TextEditor, VSBrowser } from "vscode-extension-tester"; | ||
import * as utils from './utils/testUtils'; | ||
import * as constants from './definitions/constants'; | ||
|
||
const path = require('path'); | ||
const assert = require('assert'); | ||
|
||
describe('LCLS tests for Gradle Project', function () { | ||
let editor: TextEditor; | ||
let actualSeverXMLContent: string; | ||
|
||
before(() => { | ||
utils.copyConfig(path.join(utils.getGradleProjectPath(), 'src', 'main', 'liberty', 'config'), path.join(utils.getGradleProjectPath(), 'src', 'main', 'liberty', 'config2')); | ||
}); | ||
|
||
it('Should copy content of server.xml', async () => { | ||
const section = await new SideBarView().getContent().getSection(constants.GRADLE_PROJECT); | ||
section.expand(); | ||
await VSBrowser.instance.openResources(path.join(utils.getGradleProjectPath(), 'src', 'main', 'liberty', 'config2', 'server.xml')); | ||
|
||
editor = await new EditorView().openEditor('server.xml') as TextEditor; | ||
actualSeverXMLContent = await editor.getText(); | ||
|
||
assert(actualSeverXMLContent.length !== 0, 'Content of server.xml is not in copied.'); | ||
console.log('Sever.xml content:', actualSeverXMLContent); | ||
|
||
}).timeout(10000); | ||
|
||
it('Should show diagnostic for server.xml invalid value', async () => { | ||
|
||
await VSBrowser.instance.openResources(path.join(utils.getGradleProjectPath(), 'src', 'main', 'liberty', 'config2', 'server.xml')); | ||
editor = await new EditorView().openEditor('server.xml') as TextEditor; | ||
|
||
const hverExpectdOutcome = `'wrong' is not a valid value of union type 'booleanType'.`; | ||
const testHverTarget = '<logging appsWriteJson = \"wrong\" />'; | ||
|
||
await editor.typeTextAt(17, 5, testHverTarget); | ||
const focusTargtElemnt = editor.findElement(By.xpath("//*[contains(text(), 'wrong')]")); | ||
await utils.delay(3000); | ||
focusTargtElemnt.click(); | ||
await editor.click(); | ||
|
||
const actns = VSBrowser.instance.driver.actions(); | ||
await actns.move({ origin: focusTargtElemnt }).perform(); | ||
await utils.delay(5000); | ||
|
||
const hverContent = editor.findElement(By.className('hover-contents')); | ||
const hverValue = await hverContent.getText(); | ||
console.log("Hover text:" + hverValue); | ||
|
||
assert(hverValue.includes(hverExpectdOutcome), 'Did not get expected diagnostic in server.xml'); | ||
|
||
editor.clearText(); | ||
editor.setText(actualSeverXMLContent); | ||
console.log("Content restored"); | ||
|
||
}).timeout(35000); | ||
|
||
it('Should apply quick fix for invalid value in server.xml', async () => { | ||
const section = await new SideBarView().getContent().getSection(constants.GRADLE_PROJECT); | ||
section.expand(); | ||
await VSBrowser.instance.openResources(path.join(utils.getGradleProjectPath(), 'src', 'main', 'liberty', 'config2', 'server.xml')); | ||
|
||
editor = await new EditorView().openEditor('server.xml') as TextEditor; | ||
const stanzaSnipet = "<logging appsWriteJson = \"wrong\" />"; | ||
const expectedHoverData = "<logging appsWriteJson = \"true\" />"; | ||
await editor.typeTextAt(17, 5, stanzaSnipet); | ||
await utils.delay(2000); | ||
const flagedString = await editor.findElement(By.xpath("//*[contains(text(), '\"wrong\"')]")); | ||
await utils.delay(7000); | ||
|
||
const actions = VSBrowser.instance.driver.actions(); | ||
await actions.move({ origin: flagedString }).perform(); | ||
await utils.delay(3000); | ||
|
||
const driver = VSBrowser.instance.driver; | ||
const hoverTxt = await editor.findElement(By.className('hover-row status-bar')); | ||
await utils.delay(2000); | ||
|
||
const qckFixPopupLink = await hoverTxt.findElement(By.xpath("//*[contains(text(), 'Quick Fix')]")); | ||
await qckFixPopupLink.click(); | ||
|
||
const hoverTaskBar = await editor.findElement(By.className('context-view monaco-component bottom left fixed')); | ||
await hoverTaskBar.findElement(By.className('actionList')); | ||
await utils.delay(2000); | ||
|
||
const pointerBlockedElement = await driver.findElement(By.css('.context-view-pointerBlock')); | ||
// Setting pointer block element display value as none to choose option from Quickfix menu | ||
if (pointerBlockedElement) { | ||
await driver.executeScript("arguments[0].style.display = 'none';", pointerBlockedElement); | ||
} else { | ||
console.log('pointerBlockElementt not found!'); | ||
} | ||
const qckfixOption = await editor.findElement(By.xpath("//*[contains(text(), \"Replace with 'true'\")]")); | ||
await qckfixOption.click(); | ||
|
||
const updatedSeverXMLContent = await editor.getText(); | ||
await utils.delay(3000); | ||
console.log("Content after Quick fix : ", updatedSeverXMLContent); | ||
assert(updatedSeverXMLContent.includes(expectedHoverData), 'Quick fix not applied correctly for the invalid value in server.xml.'); | ||
|
||
editor.clearText(); | ||
editor.setText(actualSeverXMLContent); | ||
console.log("Content restored"); | ||
|
||
}).timeout(38000); | ||
|
||
after(() => { | ||
utils.removeConfigDir(path.join(utils.getGradleProjectPath(), 'src', 'main', 'liberty', 'config2')); | ||
console.log("Removed new config folder:"); | ||
}); | ||
|
||
}); |
Empty file.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is
hoverTaskBar
required? There is no actionable task performed byhoverTaskBar
. It seems that removing below two lines does not affect the test. Kindly confirm.