Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 5 additions & 7 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,18 +14,16 @@ jobs:
strategy:
matrix:
os: [ubuntu-latest]
emacs_version: ['24.4', '24.5',
'25.1', '25.2', '25.3',
'26.1', '26.2', '26.3',
'27.1', '27.2',
emacs_version: ['27.1', '27.2',
'28.1', '28.2',
'29.1',
'29.1', '29.2', '29.3', '29.4',
'30.1',
'snapshot']
include:
- os: macos-latest
emacs_version: '29.1'
emacs_version: '30.1'
- os: windows-latest
emacs_version: '29.1'
emacs_version: '30.1'

steps:
- name: Set up Emacs
Expand Down
5 changes: 3 additions & 2 deletions test/flycheck-eldev-test.el
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
;; -*- lexical-binding: t; -*-
(require 'flycheck-eldev)
(require 'ert)
(require 'dash)
Expand Down Expand Up @@ -148,15 +149,15 @@
;; When checking the faulty `Eldev-local', we must not use it for initialization.
(ert-deftest flycheck-eldev-faulty-eldev-local-2 ()
(flycheck-eldev--test-with-temp-file "project-a/Eldev-local"
(insert "this-variable-certainly-doesnt-exist")
(insert ";; -*- lexical-binding: t; -*-\nthis-variable-certainly-doesnt-exist")
(flycheck-eldev--test "project-a/Eldev-local"
(flycheck-eldev--test-recheck)
(flycheck-eldev--test-expect-errors '(:matches "this-variable-certainly-doesnt-exist")))))

;; Test that `flycheck-eldev' really works on Eldev files if those are byte-compilable.
(ert-deftest flycheck-eldev-suspicious-eldev-local-1 ()
(flycheck-eldev--test-with-temp-file "project-a/Eldev-local"
(insert "(defun just-for-testing () (function-with-this-name-certainly-doesnt-exist))")
(insert ";; -*- lexical-binding: t; -*-\n(defun just-for-testing () (function-with-this-name-certainly-doesnt-exist))")
(flycheck-eldev--test "project-a/Eldev-local"
(flycheck-eldev--test-recheck)
(flycheck-eldev--test-expect-errors '(:matches "function-with-this-name-certainly-doesnt-exist")))))
Expand Down
2 changes: 1 addition & 1 deletion test/project-a/project-a.el
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
;;; project-a.el --- Test project with one dependency
;;; project-a.el --- Test project with one dependency -*- lexical-binding: t; -*-

;; Version: 1.0
;; Homepage: https://example.com/
Expand Down
1 change: 1 addition & 0 deletions test/project-a/test/project-a.el
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
;; -*- lexical-binding: t; -*-
;; Using loading root to test that `flycheck-eldev' handles this too.
(require 'test-util)

Expand Down
Loading