Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix(fileserver):wrong url href of displayed files #426

Merged
merged 5 commits into from
May 22, 2019
Merged

fix(fileserver):wrong url href of displayed files #426

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
59f6c41
fix(fileserver):wrong url href of displayed files
yuqingc May 21, 2019
4f19439
fix error
yuqingc May 21, 2019
eab8ebf
test:add test case for file server
yuqingc May 22, 2019
7772fc1
wip: add console to debug in azure. do not merge
yuqingc May 22, 2019
cb76563
fix win test failure
yuqingc May 22, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
Prev Previous commit
Next Next commit
wip: add console to debug in azure. do not merge
  • Loading branch information
@yuqingc
yuqingc committed May 22, 2019
commit 7772fc167530e1a1de1ccfb94a8d2246cc1dfb29
3 changes: 2 additions & 1 deletion http/file_server_test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,8 @@ test(async function serveDirectory(): Promise<void> {
assert(res.headers.has("access-control-allow-headers"));
const page = await res.text();
assert(page.includes("azure-pipelines.yml"));
assert(/<td class="mode">\([a-zA-Z-]{10}\)<\/td>/.test(page));
// assert(/<td class="mode">\([a-zA-Z-]{10}\)<\/td>/.test(page));
assert(/<td class="mode">\(unknown mode\)<\/td>/.test(page));
assert(
page.includes(
`<td><a href="/azure-pipelines.yml">azure-pipelines.yml</a></td>`
Expand Down