mirror of
https://github.com/gohugoio/hugo.git
synced 2024-11-29 21:02:07 -05:00
b017f7cb01
Currently, Hugo does not inject `livereload` script if html does not contain `<head>` or `<body>`. This sometimes happens if you create new sites without `theme` and it is hard to catch the cause soon.
This PR:
* Inject livereload script even if html does not include `<head>`, `<body>`, or `<html>`
- Modern browsers execute scripts even if they are outside `<html>`
- Some js frameworks (confirmed with vite) inject HRM script without `<html>` tag
* Append warning script to html if `<head>` or `<body>` is not in html
* Fix bug that livereload cannot be appended to the tags with attrs
Close #10105
72 lines
2.1 KiB
Go
72 lines
2.1 KiB
Go
// Copyright 2018 The Hugo Authors. All rights reserved.
|
|
//
|
|
// Licensed under the Apache License, Version 2.0 (the "License");
|
|
// you may not use this file except in compliance with the License.
|
|
// You may obtain a copy of the License at
|
|
// http://www.apache.org/licenses/LICENSE-2.0
|
|
//
|
|
// Unless required by applicable law or agreed to in writing, software
|
|
// distributed under the License is distributed on an "AS IS" BASIS,
|
|
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
// See the License for the specific language governing permissions and
|
|
// limitations under the License.
|
|
|
|
package livereloadinject
|
|
|
|
import (
|
|
"bytes"
|
|
"net/url"
|
|
"strings"
|
|
"testing"
|
|
|
|
qt "github.com/frankban/quicktest"
|
|
"github.com/gohugoio/hugo/transform"
|
|
)
|
|
|
|
func TestLiveReloadInject(t *testing.T) {
|
|
c := qt.New(t)
|
|
|
|
lrurl, err := url.Parse("http://localhost:1234/subpath")
|
|
if err != nil {
|
|
t.Errorf("Parsing test URL failed")
|
|
return
|
|
}
|
|
expectBase := `<script src="/subpath/livereload.js?mindelay=10&v=2&port=1234&path=subpath/livereload" data-no-instant defer></script>`
|
|
apply := func(s string) string {
|
|
out := new(bytes.Buffer)
|
|
in := strings.NewReader(s)
|
|
|
|
tr := transform.New(New(*lrurl))
|
|
tr.Apply(out, in)
|
|
|
|
return out.String()
|
|
}
|
|
|
|
c.Run("Head lower", func(c *qt.C) {
|
|
c.Assert(apply("<html><head>foo"), qt.Equals, "<html><head>"+expectBase+"foo")
|
|
})
|
|
|
|
c.Run("Head upper", func(c *qt.C) {
|
|
c.Assert(apply("<html><HEAD>foo"), qt.Equals, "<html><HEAD>"+expectBase+"foo")
|
|
})
|
|
|
|
c.Run("Body lower", func(c *qt.C) {
|
|
c.Assert(apply("foo</body>"), qt.Equals, "foo"+expectBase+"</body>")
|
|
})
|
|
|
|
c.Run("Body upper", func(c *qt.C) {
|
|
c.Assert(apply("foo</BODY>"), qt.Equals, "foo"+expectBase+"</BODY>")
|
|
})
|
|
|
|
c.Run("Html upper", func(c *qt.C) {
|
|
c.Assert(apply("<html>foo"), qt.Equals, "<html>"+expectBase+warnScript+"foo")
|
|
})
|
|
|
|
c.Run("Html upper with attr", func(c *qt.C) {
|
|
c.Assert(apply(`<html lang="en">foo`), qt.Equals, `<html lang="en">`+expectBase+warnScript+"foo")
|
|
})
|
|
|
|
c.Run("No match", func(c *qt.C) {
|
|
c.Assert(apply("<h1>No match</h1>"), qt.Equals, "<h1>No match</h1>"+expectBase+warnScript)
|
|
})
|
|
}
|